Deprecation warning when merging structured configs inheriting from Dict

I am moving the discussion I started in https://github.com/omry/omegaconf/issues/663#issuecomment-844628488 into a new issue
Is your feature request related to a problem? Please describe.
With OmegaConf 2.1.0.rc1 the following code raises a deprecation warning pointing to #663:
@dataclass
class Config1(Dict[str, Any]):
   x: int = 0

@dataclass
class Config2(Dict[str, Any]):
   y: int = 0

cfg = OmegaConf.merge(Config1, Config2)

Describe the solution you'd like
Based on the migration instructions in #663, it would seem intuitive (to me for the following to be allowed):
@dataclass
class Config1:
   x: int = 0

@dataclass
class Config2:
   y: int = 0

cfg = OmegaConf.structured(Config1)
with open_dict(cfg):
    cfg = OmegaConf.merge(cfg, Config2)

However, this currently fails with:
Traceback (most recent call last):
    ...
omegaconf.errors.ValidationError: Merge error: Config2 is not a subclass of Config1. value: {'y': 0}
    full_key:
    object_type=Config1

Discussion
Continuing the discussion started in #663:

@odelalleau, you can merge a dict like config onto a Structured Config, or another Structured Config of the same type or of a subclass.

Hmm, right, so actually this is even more confusing than I thought. My example above indeed still works (with the deprecation warning) if only Config2 inherits from Dict (it doesn't matter for Config1). I find this confusing because from an implementation point of view, the target is Config1 (see the code)
And the resulting object type is Config2 while I would have expected it to be Config1...
Anyway, that's not a big deal, it's just a matter of merge semantics (I got it backwards), and as long as the user-facing behavior is consistent, the underlying implementation doesn't matter much. But even if I open the second config for writing instead:
@dataclass
class Config1:
   x: int = 0

@dataclass
class Config2:
   y: int = 0

c1 = OmegaConf.structured(Config1)
c2 = OmegaConf.structured(Config2)
with open_dict(c2):
    cfg = OmegaConf.merge(c1, c2)

I still run into the same ValidationError, so my problem remains.
On the question:

What are you expecting the resulting underlying type to be?

I would say that if OmegaConf.merge(Config1, Config2) gives a config of type Config2 when Config2 inherits from Config1, then (assuming we can make it work) it should also be Config2 when it doesn't inherit from Config1.
The merge semantics would thus be: "add into Config2 the stuff found in Config1".

Thanks @Jasha10  -- I'll wait for the outcome of the discussion in #724 before giving more thought to my specific use case. I'm a bit ambivalent about your example: on one hand it makes the structure of the merge result explicit by defining Config3, but on the other hand it's probably going to confuse anyone who isn't familiar with the intricacies of Omegaconf merge mechanics. But if it works I'll be happy to use it :)

I think this is a bug; since c1 is the target in OmegaConf.merge(c1, c2), it should be required that Config1 is a subclass of Config2, not the other way around. I've opened #724.
Or at least it should be possible to do OmegaConf.merge(c1, c2) in the case where Config1 is a subclass of Config2. Otherwise, if we require that Config2 is a subclass of Config1, then every field from c1 will be overwritten by a field from c2, so there is no point in doing the merge.

No.
We are promoting c1 the type of c1 to be c2 after the merge.
This is intentional behavior. After the merge the ref type remains the same but the object type is c2 similar to what would happen if you assign c2 to c1. the object type is c2 after that.

Closing since the approach from https://github.com/omry/omegaconf/issues/724#issuecomment-846277178 should make it possible to work around this issue.

Closing since the approach from #724 (comment) should make it possible to work around this issue.

Actually I was a bit quick to hit the close button, I don't think this approach is fully satisfactory since we lose typing:
@dataclass
class Config1:
   x: int = 0

@dataclass
class Config2:
   y: int = 0

cfg = OmegaConf.structured(Config1)
cfg2 = OmegaConf.structured(Config2)
with open_dict(cfg):
    cfg = OmegaConf.merge(cfg, OmegaConf.to_container(cfg2))

#cfg.x = "hello"  # ValidationError
cfg.y = "hello"  # allowed :(

=> I still believe it'd be worth having a better solution, so re-opening
(NB: we don't necessarily need to fix it in 2.1, since one can always stick to inheriting from Dict and silence the deprecation warning)

Though it's not elegant, creating a mutual subclass preserves typing:
@dataclass
class Config3(Config1, Config2):
    pass

cfg1 = OmegaConf.structured(Config1)
cfg2 = OmegaConf.structured(Config2)
cfg = OmegaConf.merge(
    Config3, OmegaConf.to_container(cfg1), OmegaConf.to_container(cfg2)
)

# cfg.x = "hello"  # ValidationError
# cfg.y = "hello"  # ValidationError

Though it's not elegant, creating a mutual subclass preserves typing:

Oooh thanks, that's a nice trick! I think I can live with it :)


ionic5 capacitor cannot be used

ionic5 capacitor cannot be used

do you know how to get it working with capacitor.. any sample app please ?


Real world pagination example a bit limiting? No pun intended.

This is part example/feature request and question. Take your pick.
Say I want to keep state of videosByUser in my pagination reducer. Using the paginate method
from the "real world" example I would do:
videosByUser: paginate({
  mapActionToKey: action => action.payload.userId,
  types: [VIDEOS_LIST_REQUEST, VIDEOS_LIST_SUCCESS, VIDEOS_LIST_FAILURE]
})

But what happens when I want videosByUser[1].ids to include the video id after a VIDEOS_CREATE_SUCCESS action?
Won't work. That's not an action the videosByUser pagination is listening for.

The real-world example is just one possible way of doing things. If you need some other way of structuring your data, you might need to jump into more advanced techniques, such as reselect's selectors.

You may also want to look at this recent article on Advanced Redux Entity Normalization .

Thanks guys!


Bug - OTG + MicDoodleLib transformer collision

Problem
When running both MicDoodleLib (required for Galacticraft) and OTG, OTG throws an error (wrongly) stating that the incorrect version of Forge is in use. Gist of log
Expected Result
OTG should either not state that it is incompatible with this version of Forge, or make necessary transformations aware of what MicDoodleLib has done.
Solutions
This probably won't be possible to fix - if so I'm happy to close this and just have it on record that OTG and MicDoodleLib/Galacticraft are incompatible.

Hey m1cr0man, thanks for reporting this, sorry for the late reply. Looks like a problem with OTG-Core, apparently MicDoodleLib and OTG-Core are modifying the same code. May be fixable, will need to take a look. Have to finish a big new release first though, so may take a while.

HI PG85, it's all good. I think I chatted about this on Discord at one point and I was aware of the new release that was coming.

Have added some custom code for Galacticraft to OTG-Core, should be fixed, I believe for v7. Newest dev versions are available on the OTG Discord, will release v8 asap.


Can't delete an Organization that's missing a Calendar

When attempting to delete an organization that was created without a calendar Django crashes. Traceback shows it was caused by the instance.delete()  call on line 58 in organizations/views.py.
This organization was originally created before calendars were introduced to the project so this likely won't effect users but might be a good corner case to handle?

This is an issue with having an invalid database, and I don't think its something we can fix outside of dropping/recreating the database. Definitely not a client-side issue, more of an out-of-date code issue.


GalvanizeInc.GalvanizeCirrus version 2.1.1

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

I think my token didn't have enough permissions to make tags.

I will retry with a new token.


microk8s kubectl port-forward often disconnects and makes the port down

I'm port forwarding for my argo event and sensor on microk8s 1.21 :
**sudo microk8s kubectl -n argo port-forward $(sudo microk8s kubectl -n argo get pod -l eventsource-name=webhook-train-model -o name) 8085:8085 &**
on the terminal.
But they are randomly getting down. I kept the terminal open, didn't do anything, but after sometimes or somedays, i noticed the port is not listening, after running the command again, they start working. Is there any solution to prevent this random port down problem?

I am not sure what may be causing this behavior. Do you see anything in the logs of the k8s daemons? How about exposing the services you are interested in in a different way (eg via nodeport)?

Actually exposing the service via nodeport may eliminate the problem I'm having, let me try with that. i'll also look into the logs . thanks


Add Starbound support

for some reason Starbound returns the server type and environment in uppercase, so I'm converting to lowercase to work around that.

Hello, ty for the PR! I will review this as soon as i get some free time.


Authorization Header

This prevents from sending a null Auth header (for troublesome endpoints), also allows modifying the prefix, if anyone is using other than the usual Bearer

@apexJCL, @eusdima, I actually think it would be more flexible to setup something like Apollo did. What do you guys think about that?

@HofmannZ that was my intented prev commit, where we declare a middleware to handle it [like they do over here)[https://www.apollographql.com/docs/angular/recipes/authentication.html#Header], maybe my implementation was wrong, but the idea was to do it like that.

@apexJCL I think the best way to mimic that is to create a new class for HttpLink, and add the setContext as a method on that class. What do you think?
Also we've been using React-Apollo as reference for the project, so maybe it's a good idea to not mix Angular with it ;)

@HofmannZ Then maybe it'd be better to go full on the approach of links, so we can define whether the connection is made throught http or maybe a websocket, allowing for that kind of context handling.
PS: I didn't realize it was the Angular doc, but the concept it's pretty much the same (as far as I can see)

I would leave Zino in charge of this because he has a better understanding of the roadmap following apollo client.

@apexJCL Started on the full approach with links, see #36


Continue legend overflow on new line

It was brought up that some journals put restrictions on legend size. So, it would be nice to also allow legend text to continue on a new line.
We can set up a toggle on the legend that will either fit all legend text on single line (currently how its done) or continue legend text on a new line when overflow happens.

This sounds like a nice idea. If I can ask, what sort of legend text is causing the problem? I know this is a problem with long taxonomy strings (e.g. species-level ones) -- but I'm sure these won't be the only problematic cases.
From a bit of messing around with the HTML, it should be possible to insert <br> tags into the legend label text (as shown in the first two categories below), although there's probs a cleaner way to do this.


VaultWarden and backup run as non-root

Run VaultWarden as non-root user
Precondition for https://github.com/ttionya/vaultwarden-backup, backup integration, while beneficial on it's own

I like the idea of running under a security context, but I think this should be optional in the template.

I would be okey with it being an option but I want to be able to run as not-root user since our k8s do not allow containers to run as root. For me I don't see the need to be able to run it as root but there might be resaons that it should be a choice.


adding userdata to worker nodes

At the moment, the machine-controller uses a hard coded ignition/cloud-config to create new worker.
We would like to insert extra options there, as the workaround with using a daemonset and a privileged Pod is not that good.
f.e at the moment we would need to set kernel parameter wit sysctl to get rid of some inotify problems.

Complete control over user-data customization is possible when using OSM.


Vs title getting stuck midway on screen in duel

When fighting on duel maps, , there is a chance that the Vs title screen will get stuck midway in it's animation and will be stuck on your screen the whole game

For example: this pic, the whole game it was like this even when we changed maps. I think it's an issue for the game mode itself and not a specific map.

also just for the record, please search for existing bugs. This has been fixed in dev and will be pushed in rev 15

Alright, thank you


Add an option to persist the last-selected choice for the Home Feed sort

According to @CaseyNewton, Twitter's new Latest Tweets design doesn't respect your previous choice.
Could you add an option to select your Home Feed sorting option that is persistent?

I believe "Always Show Latest Tweets" is what you're looking for. Feel free to re-open if that's not the case

LOL, nice! Hopefully it'll address Casey's needs!


NVRInteractables.Deregister() implementation and/or usage example needed.

Destroying NVRInteractable-derived objects (or changing their colliders) can result in a bugged ghost entry in NVRHand.CurrentlyHoveringOver, which produces a number of buggy behaviors such as being able to grab the object from anywhere, or making the controllers vibrate perpetually (and probably more). It seems like NVRInteractables.Deregister (and the associated downstream functions in NVRPlayer & NVRHand) are made for this purpose, but it's unclear how they're intended to be used.

Whoop, will fix.


Closes #9 Add Azure functions library and remove the API.

:white_check_mark: Build mentorbot 1.0.8-olmcfxaw completed (commit https://github.com/MentorSource/mentorbot/commit/e21e54cd1e by @rosenkolev)

:white_check_mark: Build mentorbot 1.0.8-olmcfxaw completed (commit https://github.com/MentorSource/mentorbot/commit/e21e54cd1e by @rosenkolev)

:white_check_mark: Build mentorbot 1.0.10-khbghnfw completed (commit https://github.com/MentorSource/mentorbot/commit/11992fa39d by @rosenkolev)

:white_check_mark: Build mentorbot 1.0.11-xsuguupm completed (commit https://github.com/MentorSource/mentorbot/commit/3bbb75cc56 by @rosenkolev)

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8b3d8f2). Click here to learn what that means.
The diff coverage is 44.44%.

@@            Coverage Diff            @@
##             master      #10   +/-   ##
=========================================
  Coverage          ?   16.19%
=========================================
  Files             ?       13
  Lines             ?      105
  Branches          ?        0
=========================================
  Hits              ?       17
  Misses            ?       88
  Partials          ?        0

Impacted Files
Coverage Δ

...rc/MentorBot.Business/Services/CognitiveService.cs
0% <0%> (ø)

...entorBot.Business/Processors/LocalTimeProcessor.cs
0% <0%> (ø)

...entorBot.Core/Models/Options/GoogleCloudOptions.cs
0% <0%> (ø)

src/MentorBot.Core/Localize/StringLocalizer.cs
0% <0%> (ø)

...MentorBot.Business/Services/HangoutsChatService.cs
100% <100%> (ø)

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 8b3d8f2...cc23a8b. Read the comment docs.


support profile aware properties

see https://quarkus.io/guides/config-reference#profile-aware-files
you can have application-<profile>.properties now.
atm content assist and validation seems to ignore them.

If you install last vsix (not released) of vscode-quarkus, it should work, but it's not perfect on Java file side. I created the PR https://github.com/eclipse/lsp4mp/pull/180 to manage correctly profile inside Java file (for hover and diagnostics) for microprofile-config-profile.properties and  microprofile-profile.properties

Fixed with https://github.com/eclipse/lsp4mp/pull/180


Create a JASOn file with 10 jobs according to the structure to display on the homepage

https://github.com/OlushoreOdude/nyf-job-board/pull/19


Adjustments for line height in the text label

The line height was not being scaled with the font size.  This lead to the height of the actual label not changing when it was animated to the active state.  Also removed the annoying value result that raises a proptypes warning when value is empty.

@alburdette619 tests seem to be failing

The line height seems to be pushing the text a little to high on web


Frame Grabber not working with RPLidar A1M1

We've got an RPLidar A1M1 device with 1.15 firmware - too old a model to upgrade, as stated by dfu_upgrader utility (working for A1M8 only).
The ultra-simple app registers incoming data properly, i.e. the messages with theta and distance values are coming instantly. So the lidar works well.
However, nothing shows in the frame_grabber window (the command buttons seem to work). Also no firmware version seems to be registered (displaying 0 all the time).
I noticed that there's an undocumented "welcome message" sent via UART after power-up or restarting the device. It consists of 56 bytes, whereas some applications report it to be 63 in RPLidar A2 (which should have the same protocol as A1, according to the application's note). Could it be an issue here?
The "welcome message" that my lidar sends is (with CR+LF as newline characters):

RP LIDAR System.
Firmware Ver 1.15, HW Ver 0
Mode: 0

I recently bought a A1M8 and just received it. I have tried all three of the apps on three different machines:

Frame grabber
Simple grabber
Ultra-Simple
and (on all three machines) they report: 'cannot bind to the specified serial port com3'
The machine beeps when I plug/unplug, so I'm pretty sure it is recognizing the device, and the Lidar spins when I plug it in (and the green light on the bottom turns on).

Maybe the little serial adapter is bad?
Otherwise, I'll probably return the whole package.

Its been a while, but as I recall this software throws an error and looks
like dead but keeps running.  After tracking down a many years old compiler
to compile it, I finally got something that generated a point cloud (with
errors).
Like most Chinese hardware the software is buggy and pretty much unusable
with no documentation or support.
Unless you want to code to bare bones through experimentation you may want
to return it.
On Tue, Mar 19, 2019 at 2:31 AM Ed Silky<EMAIL_ADDRESS>wrote:

I recently bought a A1M8 and just received it. I have tried all three of
the apps on three different machines:

Frame grabber
Simple grabber
Ultra-Simple
and (on all three machines) they report: 'cannot bind to the specified
serial port com3'
The machine beeps when I plug/unplug, so I'm pretty sure it is
recognizing the device, and the Lidar spins when I plug it in (and the
green light on the bottom turns on).

Maybe the little serial adapter is bad?
Otherwise, I'll probably return the whole package.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Slamtec/rplidar_sdk/issues/6#issuecomment-474215272,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdOvClngKHbHnU20FvYZPkXnO8OwDe2ks5vYIRBgaJpZM4ZLJII
.

--
John Ellenberger
Groton, MA

I just recently bought an A1M8. The demo solution file throws a ton of errors for VS 2019 Community edition. Has anyone used these successfully recently?

I recently bought a A1M8 and just received it. I have tried all three of the apps on three different machines:

Frame grabber
Simple grabber
Ultra-Simple
and (on all three machines) they report: 'cannot bind to the specified serial port com3'
The machine beeps when I plug/unplug, so I'm pretty sure it is recognizing the device, and the Lidar spins when I plug it in (and the green light on the bottom turns on).

Maybe the little serial adapter is bad?
Otherwise, I'll probably return the whole package.

Hi sir
Could you solve the problem of launching Lidar?
My Lidar doesn't even know the computer port

If the computer doesnt detect the serial, then just download the manual guide at the website given.
But if the usb was already assigned to COM_ , then just run the frame grabber and mine work with 57.6 k baud rate with Windows

Hi everyone,
if you read this post, it will significate that you want to communicate with LiDAR in order to integrate it to your project. So, I thinks that my work can interest you, check it at this link : Use the LiDAR A1M8
BR,
chucknouri

Hi everyone,
if you read this post, it will significate that you want to communicate with LiDAR in order to integrate it to your project. So, I thinks that my work can interest you, check it at this link : https://github.com/NouriHuseynov/LiDAR_A1M8.git
BR,
Nouri Huseynov

Hey. I just waned to let you know that the link to your project is broken, you need to remove the %C2%A0 from the end of the link.
Sadly, I no longer have access my a RPLIDAR, or I would love to help out.

Oh sorry @Marsfan I update the link in my post, I think everything is ok now (I hope)...

@NouriHuseynov Yup. Looks good. No worries, just wanted to make sure you know about it


Propose RFC for IpAddr common methods

rendered

Please include the exact signature of the proposed methods in the proposal... something like this:
impl IpAddr {
    fn is_loopback(&self) -> bool { ... }
    fn is_multicast(&self) -> bool { ... }
}

@eefriedman Just added that.

Similar suggestion (without required details) https://github.com/rust-lang/rfcs/issues/1554 cc @canndrew

It looks like there are some other shared methods between the two types (is_documentation, etc). Seems like we should probably add them all at once?

@sfackler I only mentioned the stable methods. If it would be better to list all of them, I can make that change.

Yeah I figure we should add all of them - these new methods would land unstable anyway.

@sfackler Done.
Also, I noticed that Ipv4Addr has an is_link_local() method, while Ipv6Addr has an is_unicast_link_local() method. It seems like those are nearly the same; should I add an is_link_local() as well?

I don't think I know enough about IPv6 to be able to answer that :)

Neither do I. Unless an IPv6 expert shows up and says that those are the same, I think it's best to leave is_link_local() out.

I'm actually surprised to see this RFC because I thought these methods already exists! I could have sworn we either had PR or an issue about this...
@mathphreak this is actually the kind of addition where the libs team generally doesn't require an RFC. If you'd like to close this out and send a PR directly to rust-lang/rust proposing this change, I suspect we'd merge pretty quickly!

@alexcrichton I wasn't quite sure if that was the case. I'll get right on that. I'll leave this open until I submit that PR.

That took longer than expected, but https://github.com/rust-lang/rust/pull/34694 is now live.


Setup babel-preset-cra-universal

Followed suggestion from #128 to implement a custom Babel preset. This is a copy of babel-preset-razzle and only offers name changes to the babel-preset-razzle. Should there be any changes needed feel free to request them for this PR and I will try to get to them asap 🙂

Hi, thanks for the PR! 🎉
Since cra-universal already used Babel 7, we also must use some of their official scoped packaged
The presets and plugins used:
https://github.com/antonybudianto/cra-universal/blob/master/packages/cra-universal/src/config/server/.babelrc
Let me know if you need any help.

Hi @ajchili , any updates on this?

Hey, I still plan to get this done but got swamped with other work.
Tomorrow or Thursday I should have amendments.
On Tue, Nov 27, 2018 at 8:19 PM Antony Budianto<EMAIL_ADDRESS>wrote:

Hi @ajchili https://github.com/ajchili , any updates on this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/antonybudianto/cra-universal/pull/129#issuecomment-442281604,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHi5d88Eh1cyjO5e-8JlllVtosvsy3U0ks5uzeSvgaJpZM4YqI2Z
.

Made the requested changes. I hope I did them right. If not I may need some help as I am still learning what a babel preset is and how to create one.


API route fails when passing page or per_page parameters

I attempted to call the following URL (with a valid key):
https://metasmoke.erwaysoftware.com/api/posts/feedback/?per_page=10&type=naa-&page=1&key=<VALIDKEY>

The goal is to get the first ten results. However, when I do so, I receive the following error:
ArgumentError in ApiController#posts_by_feedback

comparison of Fixnum with String failed
Extracted source (around line #116):

114
115    def set_pagesize
116       @pagesize = [params[:per_page] || 10, 100].min
117    end
118
119    def has_more?(page, result_count)

The request being sent (from that error page, is as follows - again with a valid key):
{"per_page"=>"10", "type"=>"naa-", "page"=>"1", "key"=>"<VALIDKEY>"}

A similar error errors if I remove the per_page parameter from the URL (but leave the page parameter)
ArgumentError in ApiController#posts_by_feedback

comparison of String with 1512 failed

118
119    def has_more?(page, result_count)
120        (page || 1) * @pagesize < result_count
121    end
122
123    def verify_write_token

The request on this call is:
{"type"=>"naa-", "page"=>"1", "key"=>"<VALIDKEY>"}

If I remove both the per_page and page parameters, I receive a response, as expected

This is why we test our code, kids.
I'm pretty busy over the next three days, so I may or may not have time to fix this. I know what that cause is, but I need to run some tests ideally, which I don't have the hardware to do anytime soon. @Undo1 may be able to get to it before me (hint: the solution in has_more? isn't a to_icall, because nil || 1 == 0, or something. It breaks things, anyway.)

I may have been wrong. Both about my time and about the solution.
Sigh.


Fixed typo in custom-cms-element how-to

1. Why is this change necessary?
For a better documentation
2. What does this change do, exactly?
It changes compoent to component.
3. Describe each step to reproduce the issue or behaviour.
Add an a after the letter o in the word compoent.
4. Please link to the relevant issues (if any).

5. Which documentation changes (if any) need to be made because of this PR?

6. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

thanks @Haberkamp for your contribution :+1: :tada: :blue_heart:
will be merged soon


Create apprentice page

@Scripore:
Getting the following merge error:
error: Your local changes to the following files would be overwritten by merge:
public/blog/2015/09/30/announcing-operation-code-board-officers.html
public/blog/feed.xml
public/blog/index.html
Please, commit your changes or stash them before you can merge.
error: The following untracked working tree files would be overwritten by merge:
public/blog/2015/11/08/introducing-the-new-operation-code-board-chair.html
public/blog/2015/11/17/linux-academy.html
Please move or remove them before you can merge.
Aborting
[15:33:56


 Removing the "open browser" step

Hello, just made an account for this message. Everything is working great for me on Chrome so far. I'm using Windows 10. As a non programmer it was a little intimidating getting it set up but I quickly realized it wasn't too bad.
There's one thing I'm curious about, that being the "starting browser" step. I use the Youtube Music app, which works great with YoutubeMusicRPC, but it means I don't really need to open the chrome app directly to get things working. I don't know how important that step is for the Presence to decipher my browser data. Assuming it's supposed to just make the process easier on the user, it does the opposite in my case and for anyone else who wants to listen from the app. If there was an option to disable that step, that'd be a small but much appreciated quality of life change.
I'm just starting to use YoutubeMusicRPC so in the future I'll just close the chrome instance once it's opened. Like I said it detects the Youtube Music app without a hitch, and it's really just one extra mouse click so it's not a big issue.
If this is a confusingly worded, I'm sorry I have little understanding of how this stuff works. Cheers :)

Hi @wtolman,
Thank you very much for trusting and using the app. I am considering setting the default connection without having to open a browser tab, but I will only implement it in a new version once I ensure it is secure. I will keep you notified!


Issue: due Flag Not Functioning as Expected

Issue: due Flag Not Functioning as Expected
Description:
I am experiencing an issue with the due flag in Ultralist. When I attempt to add a due date to a task using the due flag, the due date does not appear as expected.
Steps to Reproduce:

Run the following command to add a task with a due date:
ultralist add "Learn how to conduct a Phishing Campaign +TCM-Security due:tom"

List all tasks to verify the due date:
ultralist l

Expected Behavior:
The task should be added with the specified due date. For example:
all
1  [ ]  Tomorrow  Learn how to conduct a Phishing Campaign +TCM-Security

Actual Behavior:
The task is added, but the due date is not displayed:
all
1  [ ]    Learn how to conduct a Phishing Campaign +TCM-Security
2  [ ]    Learn how to conduct a Phishing Campaign +TCM-Security

System Information:

Operating System: Linux Mint

Additional Context:
This issue persists regardless of the due date format (e.g., due:tom, due:sep1, due:2023-09-01). I have also tried enclosing the task description and due date in quotes, but the due date still does not appear.
Any assistance in resolving this issue would be greatly appreciated.
Thank you!

Dunno how, but it seems to work now,


DROP DATABASE

Problem
Sometimes the database just has to go, to be recreated from nothing. Would be nice if Prisma CLI would help me with that.
Suggested solution
prisma db drop or similar. Probably with a lot of checks that you are dropping the correct one and really, really understand what you are doing.
(On the other hand, normal credentials should not be able to do this anyway, but who knows... better safe than sorry.)
Alternatives

Use another tool
Maybe $executeRaw?

I've tried to go the $executeRaw way.
I figured to drop (using $executeRaw) and then re-create the db via prisma db push as a way to reset the db for tests.
However, the simple code like this:
await this.client.$executeRaw`DROP DATABASE IF EXISTS ${this.dbName};`

...gives me this output:
Invalid `prisma.executeRaw()` invocation:

      Raw query failed. Code: `1064`. Message: `You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '?' at line 1`

      at cb (node_modules/@prisma/client/runtime/index.js:38735:17)

prisma:query DROP DATABASE IF EXISTS ?;
prisma:query DROP DATABASE IF EXISTS ?;

I'm open to suggestions for resolving this or a better (simple) workaround, possibly without needing to add a package dependency.

@jagged3dge just to check, are you sure that this.dbName is defined here?

@Jolg42 Yes, sir. This is defnitely defined. This is part of a startup script for Jest testing-rig.
For completeness, here's the script in itself:
import { ProjectConfig } from '@jest/types/build/Config';
import { PrismaClient } from '@prisma/client';
import NodeEnvironment from 'jest-environment-node';
import { promisify } from 'util';
import { exec } from 'child_process';

const execAsync = promisify(exec);

export class PrismaTestEnvironment extends NodeEnvironment {
  dbName = 'lunr-test';
  dbURL: string;
  client: PrismaClient;

  constructor(config: ProjectConfig) {
    super(config);

    // Generate the mysql connection string for the test schema
    this.dbURL = `mysql://lunr-testr:lunr@localhost:3306/lunr-test`;

    process.env.DATABASE_URL = this.dbURL;

    this.global.process.env.DATABASE_URL = this.dbURL;

    this.client = new PrismaClient({
      log: ['query'],
    });
  }

  async setup() {
    // Drop the schema after the tests have completed
    await this.client.$executeRaw`DROP DATABASE IF EXISTS ${this.dbName};`;

    await execAsync(
      'yarn prisma db push --skip-generate && yarn prisma db seed'
    );

    await super.setup();
  }

  async teardown() {
    await this.client.$disconnect();
  }
}

module.exports = PrismaTestEnvironment;

@jagged3dge After testing this works
  await prisma.$executeRawUnsafe(`DROP DATABASE IF EXISTS ${dbName};`);

Now I'm gonna check why the first one wasn't working...!

@jagged3dge Why the first one doesn't work is actually documented here:
https://www.prisma.io/docs/concepts/components/prisma-client/raw-database-access#using-variables

You cannot pass a table or column name into a tagged template placeholder.

Since https://github.com/prisma/prisma/releases/tag/3.13.0 db execute is generally available and can be used for that:
For example:

truncating a table

echo 'TRUNCATE TABLE "Post";' | npx prisma db execute --stdin --url="$DATABASE_URL"

dropping a table

echo 'DROP TABLE "Post";' | npx prisma db execute --stdin --url="$DATABASE_URL"

await prisma.$runCommandRaw({
dropDatabase: 1,
});

await prisma.$runCommandRaw({ dropDatabase: 1 });


Retry deletion of message if deletion fails

Bug being solved:
Prior to these changes, if deleting a message threw an error, the script would stop deleting any more messages. This was inconvenient because on slower connections a deletion may time out from time to time and throw an error, which would cause the script to stop.
Changes:

Now the script tries to delete the message 5 times with an adjustable delay in-between attempts. If after 5 attempts the deletion fails, the script moves onto the next message.
An option was added to the UI to adjust the retry delay. Meaning that if deleting a message throws an error, you can chose how long the program will wait before attempting to delete it again.

Implementation:
I wrapped the code to delete a message in a do while block and used two variables to ensure the block would get run again if there was an error after trying to delete the message (for a maximum of 5 reattempts). If there was no error, the script would move on to the next message.
To ensure the script would continue working if there was an error when deleting the message, I nested the "if (!resp.ok)" statement within an if statement that wouldn't evaluate to true if resp was undefined (this fixed the script stopping if deleting the message threw an error).

This change is

This closes #132


"Myelin Imaging" --> "MyelinImaging"

never use space in folder and file names (UNIX unfriendly).
wiki here: https://github.com/neuropoly/qMRLab/wiki/file_name

actually this raises another question: don't we want to use "lower camel case" instead of "bigger camel case", i.e., inversionRecovery vs. InversionRecovery. Because user doesn't need to press "shift" each time he/she wants to call a function, and autocompletion will help doing the rest of the job.

@jcohenadad I think one of the conventions for Matlab is to have class names start with an upper camel-case (e.g. InversionRecovery), and methods/functions are lower camel case lower camel-case (e.g. getValue).
That being said, I don't think qMRLab is following a camel-case convention consistently regardless; for example, the MTSAT class is completely uppercase: https://github.com/neuropoly/qMRLab/blob/master/Models/Myelin Imaging/MTSAT.m

@mathieuboudreau got it!
wiki updated: https://github.com/neuropoly/qMRLab/wiki/file_name


Intersector method to determine Polygon area of Intersection

Intersector method for intersecting two shapes and determining the resulting overlap is only usable with the Rectangle class. I'm currently attempting to determine overlap of two rotated Rectangles which isn't viable given that Rectangles do not currently have a concept of rotation. So the only other option currently is to use a Polygon. I recognize the difficulty of Polygon clipping and the fact that tasks such as this can be solved by Box2D, but it seems beneficial to also have the functionality in vanilla LibGDX

Feel free to try a PR, else I'm not sure when someone will find time for this.

Can do. I'll see what I can whip up in the next week or so.


Incomplete SSL Tutorial and Errors When Accessing Example After Deploying onlyoffice:8.2 with Docker

Hello, I only found a "new topic" button on the official forum, but no place to ask a question directly. Therefore, I am posting my issue here.
Environment: Ubuntu 24.10 + Docker + onlyoffice/documentserver:8.2
While following the official tutorial (https://helpcenter.onlyoffice.com/installation/docs-community-install-docker.aspx) to deploy ONLYOFFICE Document Server, I encountered the following issues:
If SSL is enabled and configured as prompted, it also needs -p 443:443 to be specified, which was not mentioned in the tutorial.
After correctly configuring SSL, when using https://<IP_ADDRESS> to test access with the example, it is not possible to create or download files, and an error occurs.


Make xz-embedded build work on Glitch

https://github.com/wh0/nix-cache-view/blob/1a5bba2d55272506ff2f7c6af27cf0bf92d79474/auto.sh
The commented out part doesn't work due to wasi-sdk 16 requiring a newer glibc than is available on Glitch.
I think wasi-sdk 12 works, so let's switch to that. It requires a main function (was this a bug that they fixed later?), so we'll have to add it. It can be in a separate file so that we don't have to mess with xz-embedded's source directory.

bzip2 too now

929a21841782eb8e7d8a409bc84b267063c0d29a


Email ID Validation

Email Id is not checked for validation.
In register page I typed " hello ''  in mail column and it did not show any message to enter correct mail Id .It can be checked if email user enters is in the valid format or not

Assigning it to you. Check the same for both registration and login page.

Can i work on this issue.
I have worked with express-validator so I can use it here.

can I work on this issue?

@thebishaldeb I will fix it by today

My exams are going on so my schedule is a bit packed.Can I make contributions after 20 December

My exams are going on so my schedule is a bit packed.Can I make contributions after 20 December

Yeah cool. No problem.

@devyaniChoubey  this one is still assigned to you.

Can I work on this @thebishaldeb ?

When we click the submit button it is already showing error email not valid.

Okay, I guess I did it then. Closing the issue,


Avoid computing Transaction fees several times in mempool

Currently in the mempool code for validating a transaction, there are several calls to try_get_fee. This is wasteful as each call to this function involves a subsystem call to chainstate. Perhaps we should consider adding a TransactionWithFee struct (not the nicest name but you get the idea) which stores the fee. TxMempoolEntry stores the fee but it is created late in the validation flow, after many calls to try_get_fee have already been made.
As we've already discussed, we eventually want to have the mempool call TransactionVerifier to perform all non-mempool-related validity checks. Perhaps the output of TransactionVerifier should be some kind of VerifiedTransaction which caches the fee.

Can you please add some description and details to this?

Done


added student marks calculator project...pls review it

Related Issue

Add student marks calculator

Closes: #503
Checklist:

[✔ ] My code follows the style guidelines of this project.
[ ✔] I have performed a self-review of my own code.
[ ✔] I have commented my code, particularly in hard-to-understand areas.
[ ✔] I have made corresponding changes to the documentation.
[✔ ] New and existing unit tests pass locally with my changes.

Screenshots

@Vishal-raj-1 @urvashi-code1255  pls review my PR and guide me if any changes are needed

@bishalshaw-164 Number more than 100 can be inserted due to which percentage goes beyond 100 fix that please.

@bishalshaw-164 Number more than 100 can be inserted due to which percentage goes beyond 100 fix that please.

@bishalshaw-164 updates ?

@Vishal-raj-1  done review it

Allow to enter only number under 100

Okay wait
On Mon, 12 Apr, 2021, 5:42 pm Rajat Kumar, @.***> wrote:

[image: image]
https://user-images.githubusercontent.com/38184186/114392395-55c10b80-9bb6-11eb-8b01-4b562ec0ccaf.png
Allow to enter only number under 100
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Vishal-raj-1/Awesome-JavaScript-Projects/pull/561#issuecomment-817758753,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR5Z7BQDIU75DQET46U664LTILPSRANCNFSM4ZOXW6AA
.

@991rajat done review it 😇😇

Congratulations on merging of first PR. :tada:
Keep Contributing.

@991rajat  thanku it means a lot to me🤩🤩😍


Add data disclaimer

Add a statement of data protection to the user registration page & profile page

So this already exists now more or less in our mouseover text, and I don't think we're gonna do much better than that. Gonna mark this one ask closed as well.


textarea does not honour autosize when initial value is set via ngModel

Bug, feature request, or proposal:
When I have a textarea with "mdTextareaAutosize" and a bound "ngModel", the textarea doesn't seem to size to the initial data. Once you change anything inside the textarea, it will start autosizing.
What is the expected behavior?
When page loads, textarea should be sized appropriate to data.
What is the current behavior?
When page loads, textarea is not sized.
What are the steps to reproduce?
http://plnkr.co/edit/D9ZmsmhRMNc2rjXkmVXv?p=preview
What is the use-case or motivation for changing an existing behavior?
Applications that have dynamic data in text areas that needs to be autosized on load.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
I have only tested in Chromium 58 on Linux. For Angular, it looks like your plunker template uses 4.1.3, but in the application I discovered it, we're using 4.0.0. Typescript is 2.2.1. Material is 2.0.0-beta.2 (although your plunker uses 2.0.0-beta.5).

1 :+1:
something is wrong with mdTextareaAutosize

+1

Just ran into this.  Any thoughts on a workaround?


Permissions

I have a user with Security & Compliance Center "Security Administrator" and they cannot view or manage Anti-Phish Policies. They are unauthorized for  Request: /api/antiphishpolicy
I am having this issue across the ATP landscape as is evidenced on the Threat Management Dashboard.

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

ID: 70ea08aa-4520-8f48-93e2-0ce74cd43298
Version Independent ID: 7be23d3d-f72b-abd3-e2c4-96ff3998a59b
Content: Configure ATP anti-phishing policies - Office 365
Content Source: microsoft-365/security/office-365-security/configure-atp-anti-phishing-policies.md
Service: o365-seccomp
GitHub Login: @chrisda
Microsoft Alias: chrisda

@Breaker119, please open a support case. Security Administrator should work.

@Breaker119 - Thank you for submitting feedback.
From our understanding, the issue you raised has been answered by @chrisda so we will close this issue. You can open a service ticket in your tenant so this can get resolved ASAP.
Thank you for your contribution to make the docs better! Much appreciated!

Actually no. It's still an issue. I know have an active support case. Case #:19793769

From: Joyce Y<EMAIL_ADDRESS>Sent: Thursday, May 7, 2020 6:28:18 PM
To: MicrosoftDocs/microsoft-365-docs<EMAIL_ADDRESS>Cc: Breaker119<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/microsoft-365-docs] Permissions (#1774)
Closed #1774https://github.com/MicrosoftDocs/microsoft-365-docs/issues/1774.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/microsoft-365-docs/issues/1774#event-3315009495, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALMMMJ5ERGROUT42M4DPGZTRQM7ZFANCNFSM4MTBSQ3A.

@Breaker119 - Sorry to hear that you're having some technical issues. Please understand that this site is a product documentation site and not a support site. As such, the feedback section here is intended for feedback that improves the content in the Microsoft documentation and not intended for technical support type issues. If you do find that it is a documentation error though, feel free to comment back and we'll gladly look into fixing it.  I hope this gets you pointed in the right direction. Thank you.

Yes thank you - fast was my point in creating the issue. Following the documentation precisely did not work. The permissions outlined do not provide free necessary access to the security dashboards.
Jason Heiser

From: Joyce Y<EMAIL_ADDRESS>Sent: Friday, May 8, 2020 8:19:58 PM
To: MicrosoftDocs/microsoft-365-docs<EMAIL_ADDRESS>Cc: Breaker119<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/microsoft-365-docs] Permissions (#1774)
@Breaker119https://github.com/Breaker119 - Sorry to hear that you're having some technical issues. Please understand that this site is a product documentation site and not a support site. As such, the feedback section here is intended for feedback that improves the content in the Microsoft documentation and not intended for technical support type issues. If you do find that it is a documentation error though, feel free to comment back and we'll gladly look into fixing it. I hope this gets you pointed in the right direction. Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/microsoft-365-docs/issues/1774#issuecomment-626082294, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALMMMJ7WJPZ7U53RNZBFRHDRQSVT5ANCNFSM4MTBSQ3A.


Multiclass classification

How to make code changes to detect multiclass classification. 30 classes problem.
Is it even possible?

I think it is possible, but have no time to do it now. It's a classification first and identification second task. Currently, there is not enough work on it.

No worries, I can implement it if you can guide me through non-technical stuff.
I am planning to test it for this use case and see how it performs.

Until now, I've made a PoC to check if we can add numbers using CNN and it seems to work alright https://www.kaggle.com/c/ultra-mnist/discussion/313981

So, on a very high level can tell me what should be my steps? I will start working on it

Well, I think this task is very different from the person search task.
In your task, it's a normal object detection (with bbox, obtain it by cv2.findContours), multi-label classification (without bbox). But to be honest, I know only a little in multi-label classification, sorry for that.

We can consider this as a multiclass classification (not multilabel). It is okay Even if we are not able to get high accuracy for this problem. But the main problem that we are trying to solve is to detect bboxes of multiple objects in high resolution images - single/ multi class.

Well, I think this task is very different from the person search task. In your task, it's a normal object detection (with bbox, obtain it by cv2.findContours), multi-label classification (without bbox). But to be honest, I know only a little in multi-label classification, sorry for that.

You can think of it as a digit search task

Sorry, INF800, I have to say I have many urgents things to do :(, so I cannot help you on this topic.


Make sukebei have all ads PLUS onahole instead of only onahole

Coverage remained the same at 40.123% when pulling 5c5ef727ce2d144be822b3f04b306fb89a439fca on Kiloutre-patch-1 into 7264ab0d0a016d9d791c3baec9d9e9f9f0eaf844 on dev.

Coverage remained the same at 40.123% when pulling 445bad3eaa56a5422e6a49664d27e6b8e80ca576 on Kiloutre-patch-1 into 24c308cc994ae08b704a5b13970b6253fff6defb on dev.


Model: Peetoner NutriVision

https://www.petoneer.com/NutriVision
Is detect and add for IOT and smart live but bo way add option to Home Asssistant
{
"result": {
"active_time":<PHONE_NUMBER>,
"biz_type": 18,
"category": "sp",
"create_time":<PHONE_NUMBER>,
"icon": "smart/icon/ay1541381385946hoIoV/156360121030885b7eb80.png",
"id": "bf18e3e146dd0a86f34prj",
"ip":
"lat": "52.0794936",
"local_key":
"lon": "21.0107744",
"name": "Nutri Vision",
"online": true,
"owner_id": "18127768",
"product_id": "bmwdr0lrdhyqwktb",
"product_name": "Nutri Vision",
"status": [
{
"code": "basic_indicator",
"value": false
},
{
"code": "basic_flip",
"value": false
},
{
"code": "basic_osd",
"value": false
},
{
"code": "basic_nightvision",
"value": "0"
},
{
"code": "movement_detect_pic",
"value": "$"
}
],
"sub": false,
"time_zone": "+02:00",
"uid": "eu1604020388098rm3hC",
"update_time":<PHONE_NUMBER>,
"uuid": "tkcx733561be91fc221b"
},
"success": true,
"t":<PHONE_NUMBER>758
}

@robertbigk
The Smart Camera which's category is sp is not supported now.
We will consider adding it in the future.

Ok sorry for my ash but in the future is what time weeks months years? Ok but possible to add for now a basic switch on/off for food distribution and camera another time?

Indeed, basic (doorbell) camera support with at least movement and doorbell pressed notifications would be highly appreciated. I think a lot of Home Assistant / Tuya users are waiting for this to become available.

I don't have a lot knowledge i try to learn much I can if I can help just write me if need. If need to testing device just write me:)

When You need log for this device let me know.

Hi any hope for my device because is get lower and lower on the list:)

I subscribe to this enhancement. Would like to see it fully implemented in home assistant.


[MM-35167] Display error text under submit buttons

Summary
This PR makes it so the modal error display is placed below the submit buttons, so there is enough room for several buttons.
Ticket Link
Fixes https://mattermost.atlassian.net/browse/MM-35167
Screenshots

Before The Fix

After the fix

With lots of buttons

Release Note
NONE

@abhijit-singh Can you take a look at this PR's implementation when you have the chance? This PR is mainly just to fix the bug described in the PR description, while there is a separate effort to rework how the submit buttons are styled and laid out on webapp and mobile.


CAMEL-17503: fix websocket client unable to reconnect to Server after…

… server reboot

@oscerd how does those backport labels work? Will the bot automatic create PRs after this PR is merged or how?

@davsclaus sadly in this case won't work.
The branch needs to be on the same repository of the action and the commit should be just one.
It usally creates the PR once the PR has been merged, but in this case it shouldn't have the permissions to do it.

Ok thanks I will manually cherry pick


Implement method to apply adversarial patch onto a real-world surface

Description
This pull request adds a method to apply the patch to coordinates after perspective transformation. These coordinates can be the coordinates of a real-world, quadratic surface in the targeted image.
Thank you to @yusong-tan for sharing the code!
Fixes #853
Type of change
Please check all relevant options.

[ ] Improvement (non-breaking)
[ ] Bug fix (non-breaking)
[x] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Codecov Report

Merging #891 (c11ec02) into dev_1.6.0 (f65cbb7) will decrease coverage by 43.94%.
The diff coverage is 36.53%.

@@              Coverage Diff               @@
##           dev_1.6.0     #891       +/-   ##
==============================================
- Coverage      64.68%   20.74%   -43.95%
==============================================
  Files            176      176
  Lines          17147    17189       +42
  Branches        3176     3178        +2
==============================================
- Hits           11092     3566     -7526
- Misses          4808    13409     +8601
+ Partials        1247      214     -1033

Impacted Files
Coverage Δ

art/evaluations/security_curve/security_curve.py
0.00% <0.00%> (ø)

...cks/evasion/adversarial_patch/adversarial_patch.py
27.38% <11.76%> (-15.91%)
:arrow_down:

art/estimators/classification/blackbox.py
42.85% <46.42%> (-41.02%)
:arrow_down:

art/attacks/evasion/square_attack.py
10.47% <66.66%> (-26.67%)
:arrow_down:

art/attacks/evasion/simba.py
9.56% <100.00%> (-50.72%)
:arrow_down:

...rt/defences/detector/poison/clustering_analyzer.py
8.22% <0.00%> (-86.71%)
:arrow_down:

...ces/detector/evasion/subsetscanning/scanningops.py
11.59% <0.00%> (-85.51%)
:arrow_down:

art/attacks/evasion/carlini.py
8.40% <0.00%> (-80.87%)
:arrow_down:

...defences/detector/poison/ground_truth_evaluator.py
17.80% <0.00%> (-79.46%)
:arrow_down:

...ient_descent/projected_gradient_descent_pytorch.py
16.93% <0.00%> (-78.23%)
:arrow_down:

... and 122 more

Hi @minhitbk Thank you very much for your review! I have updated the docstrings. The current approach with the static methods aims to enable users to directly use the specific implementations.


CMake generated config files should be relocatable

Bug Report
Build Problem
It is build related, but only observable by the consumer of the CMake package.
The CMakeLists.txt in the project have install and export commands that generate CMake config files that are written to share/opentime and share/opentimelineio in the install step. The pairs of files in these directories both have an issue in that the paths they contain, specifically _IMPORT_PREFIX and those for C++ include directories, and the IMPORTED_LOCATION properties have absolute paths to the machine on which they are built.
For opentime:
The _IMPORT_PREFIX is due to the absolute destination path in https://github.com/PixarAnimationStudios/OpenTimelineIO/blob/main/src/opentime/CMakeLists.txt#L51-L53
The include and lib directories are due to the absolute paths in https://github.com/PixarAnimationStudios/OpenTimelineIO/blob/main/src/opentime/CMakeLists.txt#L44-L49.
There are equivalent statements for opentimelineio.
Our usage of thirdparty libraries is to build on canonical build machines, upload binaries to a package manager, which is then served to many clients, so relocatability is important.
My recommendation, which I've applied a patch for locally, is to make all those directory specifications relative, which in turn adds code to the generated config file to dynamically determine _IMPORT_PREFIX from the location of the config file, and all other paths are then relative to that.
Here's a relevant snippet of my patch for one of the config files for your consideration, as something that worked for us:
--- a/src/opentime/CMakeLists.txt
+++ b/src/opentime/CMakeLists.txt
@@ -41,15 +41,20 @@ if(OTIO_CXX_INSTALL)
     install(FILES ${OPENTIME_HEADER_FILES}
             DESTINATION "${OTIO_RESOLVED_CXX_INSTALL_DIR}/include/opentime")

+    # make install directories relative, so that the generated config file will be relative to the dynamic prefix
+    file(RELATIVE_PATH config_include_dir ${OTIO_RESOLVED_CXX_INSTALL_DIR} ${OTIO_RESOLVED_CXX_INSTALL_DIR}/include)
+    file(RELATIVE_PATH config_lib_dir ${OTIO_RESOLVED_CXX_INSTALL_DIR} ${OTIO_RESOLVED_CXX_DYLIB_INSTALL_DIR})
+    file(RELATIVE_PATH config_dir ${OTIO_RESOLVED_CXX_INSTALL_DIR} ${OTIO_RESOLVED_CXX_INSTALL_DIR}/share/opentime)
+
     install(TARGETS opentime
             EXPORT OpenTimeConfig
-            INCLUDES DESTINATION "${OTIO_RESOLVED_CXX_INSTALL_DIR}/include"
-            ARCHIVE DESTINATION "${OTIO_RESOLVED_CXX_DYLIB_INSTALL_DIR}"
-            LIBRARY DESTINATION "${OTIO_RESOLVED_CXX_DYLIB_INSTALL_DIR}"
-            RUNTIME DESTINATION "${OTIO_RESOLVED_CXX_DYLIB_INSTALL_DIR}")
+            INCLUDES DESTINATION "${config_include_dir}"
+            ARCHIVE DESTINATION "${config_lib_dir}"
+            LIBRARY DESTINATION "${config_lib_dir}"
+            RUNTIME DESTINATION "${config_lib_dir}")

     install(EXPORT OpenTimeConfig
-            DESTINATION "${OTIO_RESOLVED_CXX_INSTALL_DIR}/share/opentime"
+            DESTINATION "${config_dir}"
             NAMESPACE OTIO:: )
 endif()

Thanks,
Mark
Expected Behavior
I would expect a CMake package generated to be relocatable to another compatible machine and still be consumable.

Is there an earlier place in the code to apply the RELATIVE_PATH keyword, such that the OTIO_RESOLVED variables are themselves relative? Now that the cmake and setup.py scripts are cleaner than they used to be, perhaps the need for fully qualified paths is gone? I'm not sure, but think it's worth looking at.

It's an interesting question. At the time of investigation, I didn't look further, as I didn't want to break any assumptions there might be on other uses of OTIO_RESOLVED and friends. So I made the change as local as possible.


Conversion functions for ordinal numbers

To help match things like first choice <-> 1st choice.
@mcl22 had this to say about ordinals in German:
1 - 10.: erste, zweite, dritte, vierte, fünfte, sechste, siebte, achte, neunte, zehnte
For 11 - 19. you build it by number + "te" like elfte (11.), zwölfte (12.), dreizehnte (13.).

From that on you use number + "ste" like zwanzigste (20.), einundzwanzigste (21.), vierzigste (40.), hundertste (100.), tausendste (1000.), millionste (1000000.)

Bear in mind that these conversions are expensive.  Each one requires fuzzy-matching against the data set another time.

I have yet to actually encounter an issue with this.  Is anyone actually seeing this as a problem?

Going to close this and revisit if anyone ever complains about it.


[Hero] Mirror Sorceress

Name/Title: Terris, the Mirror Sorceress
Appearance: Model is based on Lich with the Frozen Star set equipped, with his frost orb being replaced with a floating shard of glass, and the stature on his back being replaced with a mirror with an ornate wrought iron trim.  Uses her glass shard as a floating sword.
Primary Attribute: Intelligence
Stats:

Strength: 19 + 1.9
Agility:  23 + 3
Intelligence: 20 + 2

Base Armor: 1
Base Damage: 25 - 30
Range: Melee (200)
Movespeed: 295
Abilities:

Divining: Gaze into Terris' mirror, revealing an area and creating illusions of any enemy heroes revealed this way.
Ability Type: Target Area
Range: 1500/2500/2500/4500/Global/Global
Revealed Radius: 800/1000/1200/1400/1600/2000
Illusion Damage Dealt: 10%/15%/20%/25%/35%/60%
Illusion Damage Taken: 400%
Duration: 6/7/8/9/11/15
Cooldown: 55/45/40/35/20/15
Mana Cost: 100

Shimmer: Teleport forward, creating illusions equally spaced between you and your destination
Ability Type: Target Point
Cast Point: 0.2
Range: 300/400/500/600/800/1000
Number of Illusions: 2/2/3/4/5/6
Illusion Damage Dealt: 25%
Illusion Damage Taken: 300%
Illusion Duration: 5/6/7/8/10/16
Cooldown: 10/9/8/8/8/6
Mana Cost: 30/45/60/75/100/160

Fracturing: When Terris takes damage broken shards of glass coalesce into an illusion of her.
Ability Type: Passive
Damage Threshold: 50
Illusion Damage Dealt: 10%
Illusion Damage Taken: 500%
Illusion Duration: 20
Cooldown: 6/4/3/2/.5/.5

Hall of Mirrors: Bends the fabric of reality, causing allied units to get stronger and enemy units to get weaker the closer they get to Terris.
Ability Type: No Target
Warp Radius: 1600
Minimum Attribute Increase/Decrease: 10%/15%/20%/25%/35%/45%
Maximum Attribute Increase/Decrease: 25%/35%/45%/55%/65%/80%
Duration: 20
Cooldown: 150/130/110/90/70/40
Mana Cost: 300/350/400/450/550/650

Possible Aghanim's Upgrades:

You may now cast spells through your illusions (The spell still goes on cooldown. When casting Shimmer, Terris teleports, not the illusion)
Hall of Mirrors is now passive (reduces attribute change by 33%)

Lore: Needs Work

If this hero used the same progression we've been using for other heroes, her illusions would deal
10/15/20/25/30/40% damage. The cooldowns should not progress beyond level 4, unless the progression of other aspects is weak. Shimmer mana cost would go 30/45/60/75/105/225, and Hall of Mirrors would go 300/350/400/500/900. There are only five levels for an ultimate.

Fixed the mana costs, illusion progression, and some of the cooldowns. If the cooldown for Fracturing didn't go down after the fourth level there'd be no point in putting more levels into it, so I left it as it is.

We aren't doing heroes right now. Closed.


Investigate CRON failure

I had set up the Github action to automatically run the config generation workflow every Wednesday at 5AM EST. Looking at the container, it doesn't seem to be running. Need to investigate why.

Yea, I'd say that would be a good time. This job also runs pretty quickly too, so I don't think it should be too heavy on anything.


Using alacritty instead of urxvt

I have tried various alterations of the code in main.py but to no sucess. I have tried
def open_editor(filename):     subprocess.run([         'alacritty',         '-e', "vim",         f"{filename}",     ])
but it seems to either not work or open an urxvt edditor. Has anyone gotten alacritty to work for this?

Never mind, the above works. I just had to reload my machine.


[SMALLFIX] Implement toString for ufsStatus

Updates toString to include all fields of the status instead of just the name

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20333/
Test PASSed.


Icon & title alignment and dynamic width?

Hi, would it be possible to get the alignment of the text and icon to always be fixed to the left - similar to how other entities work? at the moment it looks like below when they're included:

without locking the width of the bar to 65% it would end up looking like this which is totally unusable as the text can't be read and the icon becomes extremely small - if the bar could have an option to dynamically resize based on title length that would be great.:

To be fair I've only tested icon or title, not both. This would require a bit more of a rework, but shouldn't be too hard to do I guess.

I'd greatly appreciate the title and icon issue being resolved, the dynamic width of the bar wouldn't be a major concern for me, it was more of an observation that other users may desire, as you said dynamic scaling would cause different sizes which could be an issue when directly comparing but if it was an individual instance wouldn't cause an issue.

The alignment should be fixed in 1.2.1. The dynamic scaling will take a bunch more work and is something I might consider for a later release, but for now you'll have to scale them yourself.


cgo pkg-config fixes

This fixes two issues with cgo support:

In go 1.4, the pkg-config values must be passed to cgo after the -- (might be the same for go 1.5). CGO_CFLAGS seems to be ignored.
Both versions require the cflags/pkg-config to be passed to rungcc1.

Repo which replicates this issue with working output vs current gb master output: https://github.com/sheenobu/gb-cgo-bug

Thanks for your fix and the reproduction, i'll include this integration tests.

@davecheney Great, thank you.

@sheenobu please take a look at the integration test failures.

@sheenobu i'm going to take a look at this now. I've created an integration test for sdl2 and can see the problem that we're not getting the right flags out of pkg-config into runcgo1

Looks like only half of this PR was correct. Replaced by #304


add Badges in README.md

I would like to add git badges like forks, stars, watchers, pull request, issues, license, last commit, repo size, contributor to the README file. Please Assign me for this and this can be labelled as gssoc20 and easy . @sarthology @sahilit @praveenscience @jamesgeorge007

That's not a medium level task

@sarthology @sahilit @praveenscience @jamesgeorge007
Pls assign me this issue, looking forward to contribute to it


Incorrect balances shown in wallet

Hi, I recently started experimenting with Nami wallet + Sundaeswap testnet and noticed something strange. When I used the Sundaeswap faucet to fund my wallet the amounts shown are all multiplied by a million. It seems likely be a UI issue? since the Sundaeswap portfolio interface shows the expected amounts. I am using Brave Version 1.32.115 Chromium: 96.0.4664.93 (Official Build) (64-bit) on Windows 10 and Nami Wallet v2.3.2. Thanks!

Hi, thanks for letting us know.
Looks like the assets on sundae swap can be divided in fraction.
Like with ADA, where 1000000 Lovelace is equivalent to 1 ADA.
Nami actually don't know about the number of decimals the asset should have, so it simply display the raw value.
I need to check more in depth to see how we handle that on our side, and if SundaeSwap minted theses coins with any metadata specifying the number of decimals.

They haven't registered any metadata for these tokens, so no decimals. They are just dislplaying it with decimals on their website.

Ah I see. So is this just an issue with metadata information? Should this issue be closed?


Added fields not working with Unity editors drawn without IMGUI

On recent Unity versions (tested with v2022.3.10f1 on my side), Unity is no longer drawing default inspector with IMGUI (but probably with Visual Elements). Making the added fields feature no longer working.
This can be verified by enabling "Use IMGUI Default Inspector" in the Project Settings > Editor.
When enabled added fields feature is working again.
When disabled, as Unity is no longer calling DrawDefaultInspector, the harmony patch is no longer effective, and RenderNewlyAddedFields  is no longer called.
I guess an additional patch would be needed in NewFieldsRendererDefaultEditorPatch constructor to add compatibility with the new way Unity is drawing inspectors. Probably by adding a patch to CreateInspectorGUI, where you could add the creation of a new IMGUIContainer, which would call RenderNewlyAddedFields (onGUIHandler).

That makes sense, thanks for raising I'll check it out


allowing for strel_size = 0 and fixing bug in compactness shape feature calculation

Seems good to me. I cannot justify why i had the 4pi in the denominator.
For the record, we are using compacness, Q as:
Q = 4pi*A/L
where A is the area and L the perimeter.

4pi*A/L^2


Add Sprig Templating

Checklist

[x] make test-all (UNIX) passes. CI will also test this
[x] unit and/or integration tests are included (if applicable)
[x] documentation is changed or added (if applicable)

Description of change

Simply add all Sprig templating functions and start a discussion.
Closes #540

@aelsabbahy I haven't much time but this is a quick win that I can pull off.
Any particular functions you were hoping to showcase in the documentation?
I am also hoping to get an afternoon or so to play with the integration testing expansion that we have talked since this is one of these PRs that unit tests don't make much sense.

Funny, I was actually working on this last night.
I'll throw a PR out there tomorrow or the day after tagging you. Honestly, going through the sprig functions not all of them make sense for goss.

My bad I missed the assignment 😆!

Honestly, going through the sprig functions not all of them make sense for goss.

Well I thought about that as well like looking at the API... I thought about importing the Sprg functions one by one instead of pulling them all. At the same time that is more work upfront and to maintain, so just having all even if they don't make sense in the Goss context.

@aelsabbahy I will wait for your PR. Let me know if you want me to jump onto this instead.

Sorry for the delay in this. So my initial thoughts was to support a subset of the HermeticTxtFuncMap()
But honestly, at this point I'm thinking your solution is just much easier.
We should have an example file and an example_rendered version for documentation.

Sorry for the delay in this. So my initial thoughts was to support a subset of the HermeticTxtFuncMap()
But honestly, at this point I'm thinking your solution is just much easier.

Ups sorry, I shouldn't have closed it.

We should have an example file and an example_rendered version for documentation.

Ok I will get the documentation up and running and add an example based on this hello example plus a link to their documentation as well.

@aelsabbahy The documentation is been added. Please let me know what you think when you have time.
I explained the load order and that the custom Goss functions will overwrite the Sprig functions.

Great news as I too was about to look at adding Sprig but you've already done it :)

Yup, seems the next release will have a nice set of features implemented.
Thanks for all your contributions.

I'm only waiting so I can merge + cut a release.. Hopefully over this week the other outstanding issues are good to go and I can just merge/cut a release with everything in it.
If no progress on other PRs, I'll merge this one and just release this. Many thanks for the contribution.

Thank you for all your hard work and kind words. 👍

Holding off on this, since I want to group up the v0.4.0 milestone stuff together. Hoping I have time to iron out the io.reader gomega matcher before then.


[Android] Fix the error that xwalk is not initialized without loading url

Add the plugin for activity delegate into XWalkWebViewEngine.init()
instead of loadUrl to make sure activity delegate can receive the
onResume callback even if loadUrl has not been invoked.

@fujunwei Please help to review

Small nitpick: Anroid should be Android.

Thanks! Updated both of the comment and code


[DPE-2259] Switch to PPA PGB exporter

Switch to PPA prometheus exporter for pgbouncer
Tested on https://github.com/canonical/pgbouncer-operator/pull/97

LGTM postfactum. Tnx!


Adding service to search teams by its name

Coverage decreased (-0.8%) to 42.105% when pulling<PHONE_NUMBER>bf852859c5890ba8854fde649af03d on dev into b1ebdd8a7cec69c92532272fbff97a0c367f20e8 on master.

Coverage decreased (-7.05%) to 35.811% when pulling 2005d8acc4116a6a61178a33eb6d8d354369d3f9 on dev into b1ebdd8a7cec69c92532272fbff97a0c367f20e8 on master.


MB-7361 Services counseling move allowances details

Description

As a services counselor, I want to be able to view the allowance details for a move, so I can make validate the information with the service member.
Acceptance criteria:
GIVEN I am a services counselor
When I am on the move details page
THEN I am able to view the following allowance details:

Branch, rank

Weight allowance

Authorized weight

Storage in transit

Dependents

Pro-gear

Spouse pro-gear

RME (required medical equipment) - NEW

OCIE (organizational clothing and individual equipment) - NEW

Reviewer Notes
Is there anything you would like reviewers to give additional scrutiny?

Changes to AllowancesTable.jsx also affects the TXO side. Fields were re-arranged and new fields added.
New columns added to the entitlements table: required_medical_equipment_weight and organizational_clothing_and_individual_equipment.

Setup
Add any steps or code to run in this section to help others prepare to run your code:
make db_dev_e2e_populate
make server_run
make office_client_run

Code Review Verification Steps

[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.
Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change
Zeplin design mockup

Screenshots

Messages

:book:
:link: MB-7361

  Generated by :no_entry_sign: dangerJS against c523253b476d97e2da40b267d30633962fad1780


addressed TODO: made SurveyForm use api endpoint for validation

Hello @erikras,
I'm trying out this repo and noticed a TODO in the survey form. I added an api endpoint for email validation. The only issue is that the survey reducer IS_VALID_SUCCESS and IS_VALID_FAIL are never called, because the reactForm asyncValidate prevents next from being called (I think that's what's going on). It's not really an issue, but leaves code that isn't called in the repo, which isn't great for an example app. The promise middleware requires types: [initial, success, fail], so that could be changed and then remove the unused/unnecessary survey reducer cases.
What do you think?

Looks good. That TODO has been around forever. 😄


Remove fibers

Update to TypeScript 2.1.4
Remove .js files
Add tsconfig.json and transpile to ES6
Read tsconfig.json in Gruntfile.js
Remove support for Xcode 5
Merge implementation for Xcode 7 and Xcode 8
Fix Xcode 6 implementation by adding abstract methods in base class.
Remove fibers
Use sync API everywhere
Use forked version of bplist-parser in order to use sync call.
Remove support for Xcode 6
Update node.d.ts
Remove NodObjC dependency
Update sleep function
Separate entry points when the module is required and when it is used as standalone application.
This way when it is required, the help will not be printed.

TODO: Get back the "prepublish": "node prepublish.js" in package.json


Content from require with raw-loader within a component, not being loaded

I have an Icon component that loads SVG icons dynamically based on a prop.
To load the SVG icon, it uses something like:
<span v-html="svg" />

And the prop is the icon name, and the value of svg above is a computed property:
svg() {
   const svg = require(`!raw-loader!./../../assets/svg/${this.icon}.svg`);
   return svg.default;
}

Now this works perfectly fine as a component, but I am completely unable to test it as the value of svg is always returned as undefined.
I am just wondering how I could go about getting this to work?
I have tried using moduleNameMapper in the jest.config.js file, to no avail.
Are there any pointers you could give me? Many thanks in advance!

Hi, thanks for the reply.
I managed to get it working in the end, although I had to use transform rather than moduleNameMapper. Now All I need to do is figure out how I can make XMLSerializer available in a test...
Will close this now though, thank you!


The progress bar on the front end of the installation plugin is still spinning

Describe the bug
When installing the plug-in, the front-end icon rotated for a long time, and refreshing the page found that the plug-in had been successfully installed.
To Reproduce

Enter the front-end management and click addons
Find the fluxcd plugin to install
Wait for the icon in the lower right corner to rotate
Refresh the page

Expected behavior
When installing the plug-in, if the icon in the lower right corner stops rotating if the installation is successful, or there is a reminder.
Screenshots

KubeVela Version

Cluster information
1.74
Additional context

The detail of addon enable progress can be seen in the Application page. And the helm type appears doesn't mean the addon's ready. In addon mechanism, we dispatch the X-Definitions first, then other resources like some Controller.
But I think it's better to add a link beside the Enable button to lead user to the application page.

There is relative plan kubevela/velaux#699

Hey @chivalryq can i work on this Issue

Hi @punithnayak. Sure. Before you start. You can first implement a button Detail which links to application page, which is at applications/addon-foo.
After that if you still have interest you can try to implement addon page. But this can be harder and more complex.


CI/CD Infrastructure Updates

GHA and Terraform Infrastructure work required to enable CI/CD per this plan. Goal is to have main deployed to staging env, with all tests run there, and then provide single-command invocation of a prod deploy, replacing all of the manual deployment steps that engineers currently follow.  (Note that dev environment will be maintained, but only used for Terraform testing, and not part of the CI/CD pipeline.)
This epic covers the following work: [TODO: split into issues]

[ ] Auto-deploy to staging env instead of dev (minor GHA changes)
[ ] Run e2e tests on staging with full data mirroring
[ ] Re-instate merge queues. This is expected to become more useful as commit rate to main increases. Possibly wait to see how much conflict we have on parallel PRs/commits to main
[ ] On-demand trigger of prod deployment via cmd that invokes via GHA workflow dispatch (can use same branch merge strategy, without a PR, possibly with a script now)
[ ] Move Zenhub issues to closed after prod deploy
[ ] Automate verification of prod deployment sha1 (replace the API-based version check with a more reliable taskdef query)
[ ] E2E tests should log the sha1 of the deployment under test
[ ] Automate logging of deployments with change log (git commit logs) to Github Releases (not just Slack).

Hey team! Please add your planning poker estimate with Zenhub @atolopko-czi @Bento007 @danieljhegeman @ebezzi

shouldn't we split into issue first?

Yeah that would definitely help. I think it's more than 10 points total lol
Do you have time to take a stab at splitting the issues, or do you want me to give it a shot? I took a look earlier today and felt like I didn't have enough context to group/split the issues 😆

I'll take a crack at it.

Amazing thanks so much, Trent!
On Fri, Aug 11, 2023, 2:01 PM Trent Smith @.***> wrote:

I'll take a crack at it.
—
Reply to this email directly, view it on GitHub
https://github.com/chanzuckerberg/single-cell-data-portal/issues/5267#issuecomment-1675398350,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQEOW7PPTGE2AKBP76YLELXU2MRNANCNFSM6AAAAAA2OX4MC4
.
You are receiving this because you commented.Message ID:
@.***>

All issues now addressed or moved to follow-up epics. Closing


[Event Request] Codeunit 99000774 "Calculate Routing Line" - OnAfterSetCalendarEntryDateFilterOnReturnNextCalendarEntry

Describe the request
I want to remove additional calendar entry filters. Therefore, I request the following event:
procedure ReturnNextCalendarEntry(var CalendarEntry2: Record "Calendar Entry"; OldCalendarEntry: Record "Calendar Entry"; Direction: Option Backward,Forward)
begin
    CalendarEntry2 := OldCalendarEntry;
    CalendarEntry2.SetRange(Date, CalendarEntry2.Date);
    // NEW BEGIN
    OnAfterSetCalendarEntryDateFiltersOnReturnNextCalendarEntry(CalendarEntry);
    // NEW END
    if Direction = Direction::Backward then begin
        CalendarEntry2.Find('-');           // rewind within the same day
        // ...
end;

[IntegrationEvent(false, false)]
local procedure OnAfterSetCalendarEntryDateFilterOnReturnNextCalendarEntry(var CalendarEntry: Record "Calendar Entry")
begin
end;

Additional context
I want to remove a calendar entry filter (for "Capacity (Effective)").
Internal work item: AB#544575

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 23210.


Oauth Prompt: Bug in try...catch block in recognizeToken function

Versions
What package version of the SDK are you using.
v4.5.0
What nodejs version are you using
v8.9.4
What os are you using
MSFT Windows
Describe the bug
App terminates with an "Unexpected token error (this token ' { ')" on line 235 in the oauthPrompt.js during the oauth flow.
Expected behavior
App to start and stay alive.
[bug]

I am facing the same issue and not able to run the bot. We are completely blocked due to this error.
SyntaxError: Unexpected token {
at createScript (vm.js:80:10)
at Object.runInThisContext (vm.js:139:10)
at Module._compile (module.js:607:28)
at Object.Module._extensions..js (module.js:654:10)
at Module.load (module.js:556:32)
at tryModuleLoad (module.js:499:12)
at Function.Module._load (module.js:491:3)
at Module.require (module.js:587:17)
at require (internal/module.js:11:18)
at Object. (D:\home\site\wwwroot\node_modules\botbuilder-dialogs\lib\prompts\index.js:19:10)
Mon Jul 15 2019 12:00:20 GMT+0000 (Coordinated Universal Time): Application has thrown an uncaught exception and is terminated:
D:\home\site\wwwroot\node_modules\botbuilder-dialogs\lib\prompts\oauthPrompt.js:235

I reverted back to v4.4.0 for the botbuilder-dialogs package as I wait for the solution. App runs as expected.

Yes, I also reverted to v4.40 for botbuilder-* package as I wait for a solution. App run as expected now.  When can we expect the fix?

I am facing the same issue, I reverted the botbuilder-dialogs package to 4.4.0.

I submitted a PR for this (linked above). I'd imagine it will go in the next release.

I'm caught up on this issue now. I see @mdrichardson PR, and am evaluating.
We should be able to get this fixed+released shortly.

Status update:
The PR has been merged to master.
The PR has been picked to the 4.5 Branch.
The 4.5.2 JS Build is done and packages are found on our build server.
I'm working on regression testing the build, and will push to NPM once that's done.

The 4.5.2 SDK has been pushed to NPM. As far as I know, this bug is now fixed, and I can close the issue.


Use SecKeyGetBlockSize instead of kSecAttrKeySizeInBits

Checklist

[x] I've run tests to see all new and existing tests pass
[x] I've followed the code style of the rest of the project
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary

If you've made changes to gyb files

[ ] I've run .script/generate_boilerplate_files_with_gyb and included updated generated files in a commit of this pull request

Motivation:
Security.framework has change to return the exact bit count of the key size instead of the block size in bytes in macOS 14 beta and aligned releases. However, BoringSSL and the previous implementation relyed on that.
SecKeyCopyAttributes  is also rather expensive to call and keySizeInBits is accessed immediately (twice) after initialisation.
Modifications:
call SecKeyGetBlockSize(_:)`` instead of looking up the kSecAttrKeySizeInBits` in the attributes dictionary.
Result:
test pass again and allocations are reduced.

@swift-server-bot add to allowlist


Getting Started with Node.js

We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner. Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for an articles to be reviewed and/or published.
Be sure to visit our Resources Page for tools, resources, and example articles to go over.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
NOTE: We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
We're looking for the first 2-3 paragraphs of the article that appropriately summarize what your article will be about.
Proposed title of article
Getting Started with Node.js
Introduction paragraph (2-3 paragraphs):
Node.js is an open-source, cross-stage, JavaScript runtime environment, based on Chrome's V8 JavaScript engine. The purpose of this engine is to compile JavaScript functions into machine code. It is well-known for its high speed and constantly advancing performance.
In this article, We will understand better what node.js is, how to install it in our machines, as well as write our first simple web server application. We shall look at each line of our web server application.
Key takeaways:
At the end of this article, the reader will be able to:

Install and set up node.js
Write a simple web server application using Node.js
Understand the basic syntax of a Node.js application
Understand each line of code that is, HTTP Server, function and parameters req and res, Status code, Content-Type, Finishing the request, Listening to the port

References:
N/A

🤔  @Terripha
Just want to make sure this topic doesnt overlap with this.
https://www.section.io/engineering-education/nodejs-os-module/

@hectorkambow  My article will focus more on understanding a simple web server application like 'hello world'. We will understand each line of code that is, HTTP Server, function and parameters req and res, Status code, Content-Type, Finishing the request, Listening to the port


Bower installation

Hi folks,
Any chance of a bower installation for Marzipano? It behaves reasonably well with bower install "git://github.com/google/marzipano" if you download a built version of the library and drop "main": "build/marzipano.js" into the .bower.json, but that feels a lot like cheating.
Alternatively any advice on how to hook it up so it builds with my standard gulp build?
... or should I just give up and push a built version through in the gulp file?
Thanks!
Roland.

A Bower package is only a reference to whatever is in the repository at a given tag, so publishing to Bower would require checking the builds into the repository (which we would prefer not to do).
Unless your project is already using npm as the package manager (in which case you can simply require Marzipano as a module), I'm afraid you'll have to vendorize the build.

Thanks. Could you expand on "vendorize"? It's not a term I'm familiar with, and Google wasn't a lot of help.

Oh, sorry. Vendorizing just means copying a library into your project's
directory (as opposed to using some kind of package manager to take care of
dependencies for you).
On Sep 24, 2016 6:54 PM, "kingrollo"<EMAIL_ADDRESS>wrote:

Thanks. Could you expand on "vendorize"? It's not a term I'm familiar
with, and Google wasn't a lot of help.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/marzipano/issues/39#issuecomment-249397794,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEKunanT5kDuJHujIv6dgBUkdo6z5d5zks5qtdQ8gaJpZM4KFhR8
.

No worries - thanks again.


Fix CI on MacOS

Now the build will pass but some tests are still failing on MacOS.
Let's try to fix these after @188 is merged.

Codecov Report

Merging #191 (393008d) into master (ee1daf5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #191   +/-   ##
=======================================
  Coverage   89.67%   89.67%
=======================================
  Files          37       37
  Lines        1598     1598
=======================================
  Hits         1433     1433
  Misses        165      165

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 ee1daf5...393008d. Read the comment docs.


Allow encryption as a configurable for storageclass creation

Allows "encryption" as a configurable in the clusterspec.yml for storageclass creation (defaults to true)
Eg:
  storage:
    type: gp2
    storage-class-provisioner: kubernetes.io/aws-ebs
    encrypted: "true"

This fixes: https://github.com/upmc-enterprises/elasticsearch-operator/issues/196


[APEXCORE-332][WIP] Improve partitioning to support partitioning other than powers of 2

Updated the Partitioner to store partition definitions as Sets of Ranges which define which values may be accepted by what ports
This substantially simplifies both the creation of new partitioning schemes (e.g. evenly distributing values) and the logic around assigning values to partitions (no confusing mask, simply a Range.contains() check)

This is still a work in progress but given that it's touching a fairly intimate piece of the core code it would be great to get some other eyes on this.

@ilganeli This is a quite disruptive change and breaks backward compatibility. I think it will be better to first discuss approach and overall direction on the mailing list before spending too much effort on coding.

Makes sense / that's why I didn't want to dive too deep. Just enough to get a feel for what it would look like.

@ilganeli : As @tweise  mentioned please start a discussion on @dev


Prepare fastq file of mapped reads for each pathogen after mapping against pathogen genomes

Description of feature
The BAM file with mapped reads from the pathogen screening step contains all pathogens. We need to prepare a separate FASTQ file for each pathogen.

Solved in the consensus PR: https://github.com/genomic-medicine-sweden/metaval/pull/31


Revamp Python API as pure-Python wrapper around polars, arro3, and geoarrow-rust-core

After a bit of exploration on this, I don't think it's worth going forward with this approach. The hope was that we could provide a relatively simple pure-python wrapper that adds extra metadata on the python side, but I don't think that will be feasible because dealing with a subclass of polars.DataFrame will be an endless battle:

polars explicitly does not support subclassing of DataFrame and Series. So if you add extra metadata onto GeoDataFrame, that metadata will get lost on every non-overridden method, because those will return a DataFrame. So we'd have to override every single polars method in the subclass to return a GeoDataFrame, and also keep track of which methods might adjust the schema.
We need some sort of way to persist metadata in polars. Dataframe-level metadata is still incomplete because methods like df.rename will change the content of the columns without changing the top-level metadata.
Really the only viable option (in my opinion) is to have extension types supported by polars, and it seems they have no interest in doing that. See https://github.com/pola-rs/polars/issues/9112 and https://github.com/pola-rs/polars/issues/7288#issuecomment-1957537542

Great to see your development in GeoArrow and support for arrow extention types in Polars is on the roadmap. So GeoPolars will exist someday :)
In the meantime, packages like polars-st have popped up as self labelled tempoarry solutions to the spacial data in polars problem.
Were there any intentions to standardizing an API for GeoPolars itself to make a transition from an alternate solution easier? I am guessing it would be hard to plan an API while GeoArrow and extension types are still under development.

I don't think it's possible to standardize an API now. It depends partially on what the polars integration APIs look like if and when they enable the features we need.


Weapon Damage 'NumericInputIcon' can only handle 'D6' dice

That is how they are initialised.The documentation of 'ShinyWidgets'does not seem to allow to change te icon with 'updateNumericInputIcon'.
See numericInputIcon("DamageDieCount", i18n$t("Dice 4 Damage"), value = 1L, min = 1L, width = "100%", icon = list(NULL, "W6"))in "tab_combat_ui.R".

Added a feature request to shinyWidgets (see https://github.com/dreamRs/shinyWidgets/issues/293)


[FEATURE] SfProductCard - add extra slot for additional content

Describe how this feature will help us improve
When you need to add additional line of text styled differently than product title you have to use extra styling.
Is your proposal related to a problem?
I tried to add category of product in SfProductCard, which is styled differently above title. I had to use :before psedoelement.
Describe the solution you'd considered
Add additional empty slot above title slot. It can be named 'subtile', 'additional-content' or 'product category' , wdyt?
<slot name="subtitle" v-bind="{ subtitle, link }">
      <SfLink :link="link" class="sf-product-card__link">
        <h4 class="sf-product-card__subtitle">
          {{ subtitle }}
        </h4>
      </SfLink>
 </slot>

For such stuff we should have a naming convention like extra-top extra-bottom(feel free to propose sth different). I see that useful in many components.


microbench module is subject to API compatibility validations

Expected behavior
All changes in the microbench module should not be subjected to API compatibility validations.
Actual behavior
Some changes in the microbench module are subjected to API compatibility validations.
Steps to reproduce
Remove a method from a benchmark class that's not in the io.netty.microbench package and run mvn clean install -am -pl microbench. This will fail on compatibility issues.

@amirhadadi that sounds like a bug...

@normanmaurer sounds like a bug to me too.
There are a couple of ways to fix it:

Move all existing classes into the io.netty.microbench package.
Override japicmp-maven-plugin and revapi-maven-plugin for the microbench module.

Which do you think is better?

@amirhadadi lets add the override


Need help from Telugu speaking

Need help to create a redlist from calculated stopwords. When stopwords are calculated, quite a few of them are not stopwords. Theese will be added to a redlist to permanently stay out of the stopword list.

hello, I'm Nikhil here.... Do u know telugu

I don't, but I was going to create a stopword-list for Telugu, so I need a little help to verify the results

My native language is Telugu. May I help you?

What I need help to is to just go through a list of possible stopwords and tell me me which aren't actual stopwords. It' doesn't require any special knowledge outside being fluent in the language.

Hi @eklem I'm Imran. I'm a native Telugu speaker. I'll Help if possible.

Great @HumbledImran ! I'll start setting up a crawler and get back when I have content that needs verification.


Performance discrepancy between local runs and hugging face demo

Hello, first of all, thank you for sharing your amazing work. However, I've noticed a difference in the results between the Hugging Face demo and running the demo.py code from the cloned GitHub repository. Even when using the same images, the performance on the Hugging Face demo appears to be much better. Could you possibly share the config settings used for the Hugging Face demo?
One change I made was to disable augmentation; I set it to accept the original image sizes directly. This adjustment improved the performance of demo.py slightly, though it still doesn't match the Hugging Face demo. It seems all the pretrained models you provided perform differently compared to HF.
I'm quite frustrated because the performance locally doesn't match the impressive results of the Hugging Face demo. I haven't modified your code except for the later tweak with the augmentation. I'm wondering if you might know why there is such a significant performance difference with the same images on the Hugging Face demo. Thanks!

Hi @vietpho, Could you share the task you are testing and your results? The HF demo code is the same as the GitHub demo code. I don't remember making any changes.
The augmentation is happening in the HF demo as well.
https://huggingface.co/spaces/shi-labs/OneFormer/blob/main/demo/defaults.py#L70


TLS 1.3 Support

Hello,
I need TLS 1.3 Support for the android application. Riot for windows seemingly supports this but android doesn't. When will TLS 1.3 be available?

Android 9

I just found out that Android does not support TLS 1.3, issue resolved.

fwiw, my lineageos 16 (android 9) does support tls 1.3 it seems

Well then the samsung firmware sucks


Bump all dependencies to latest versions

Foundational work to bring the server to the latest state in terms of external dependencies

Jitpack seems to be down currently, the pipelines will fail.


SchemaAST plugin. write schema with directives

Is your feature request related to a problem? Please describe.
Currently schema-ast plugin uses printSchema from graphql.
That function strips all directives from generated schema.
So it's impossible to use schema-ast to write down full schema to, for example, compare changes with tools like graphql-inspector
Describe the solution you'd like
An option to the schema-ast plugin or another plugin to write schema with directives (e.g., to use printSchemaWithDirectives from graphql-toolkit)

Fixed in 1.4.0 🎉


Environmental variables are not set after issuing "tsu"

I have LineageOS 17.1 on Samsung Galaxy S4 LTE.
The latest update of tsu to version 8.5.1 broke my usual workflow.
Now, when I start tsu in Termux, I don't get my fancy command prompt ($PS1 is not set) but just one symbol "#". When I run "vim somefile.txt" inside of tsu, vim doesn't have any of the settings from my .vimrc file and .vim directory. Actually, all settings from my .bashrc are not used either, e.g. the variable $EDITOR is not set.
The variable $HOME inside of tsu is /data/data/com.termux/files/home/.suroot.
I don't know what more details to provide to make the description of the problem clearer. As far as I understand, tsu doesn't copy my environment and uses some default one.

The root user now has a seperate HOME directory to prevent file permission issues on normal Termux user.
For now, you would need to copy these dotfiles into roots $HOME directory.

Thank you. I've copied all of .vim* files and directories as well as .bashrc into .suroot/ and now everything works as expected.


Add altitude correction UI field

Height of antenna can be different from height of GPS

Closed in release 2.3.0


issue with emoji_dict

for the slash branch i'm getting the following error: AttributeError: 'AutoShardedBot' object has no attribute 'emoji_dict'
so I need to find what I should I replace emoji_dict with. i'm not sure what it output so idk what to replace it with.
it still run fine without issues.
error message(it in poll.py and on line 1114):

was fix in this commit: https://github.com/RJ1002/pollmaster/commit/8666d34478a1bc9469c07a025f35aeab53cd7813


fixing sequence of Arabic days

the sequence of days doesn't match the real day of the week. so it needs to be updated as the same locale file of Tunisia moment/locale/ar-tn.js

Wrong files modified, needs to be the /src/locale folder

updated /src/locale  @  #3352