Methods in HeroService

HeroesComponent calls getHeroes() from HeroService.  There is no "getHeroes()" method in HeroService.  HeroService does have a method getHero(id: number).

The service contains that method
@Injectable()
export class HeroService {
  .......
  getHero(id: number) {
    return Promise.resolve(HEROES).then(
      heroes => heroes.filter(hero => hero.id === id)[0]
    );
  }
}

Thank you;  I did see that later.  I find the tutorial challenging to follow. Often we are told to execute a command or
add lines of code but not where we should be ( which directory or which class even ) when taking these actions.
It helps when the final code is included in the tutorial; but this did not happen for many files in the Service and Routes lessons.
I’m still trying to get those lessons to execute without success.
Thank you for your reply,
Betty
From: ercpereda<EMAIL_ADDRESS>Sent: Tuesday, April 26, 2016 11:02 AM
To: johnpapa/angular2-tour-of-heroes
Cc: Schwartz, Betty J (IS); Author
Subject: EXT :Re: [johnpapa/angular2-tour-of-heroes] Methods in HeroService (#60)
The service contains that method
@Injectable()
export class HeroService {
.......
getHero(id: number) {
return Promise.resolve(HEROES).then(

  heroes => heroes.filter(hero => hero.id === id)[0]

);

}
}
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://github.com/johnpapa/angular2-tour-of-heroes/issues/60#issuecomment-214811834

closing since this was rewritten with the CLI


problem on Google Chrome localhost only

Hello,
First of all, I'd like to point that Sidr is an excellent plugin. So I congratulate the author :+1:
I've been using the 1.1.1 version without any major problem.
Now I'm testing the 1.2.1 package and so things are quite annoying.
1.2.1 Package is installed on the same localhost as 1.1.1
But while using it on localhost with Chrome browser. Nothing works.
It tested it locally on Firefox, IE and Safari and it works fine.
It tested it remotely on Firefox, IE and Safari, Chrome Android and it works fine too.
every extensions are disabled.
So why doesn't it work locally on Chrome ?
Javascript console is silent, no error reported :-(
My Chrome version is : 31.0.1650.57 (Build officiel 235101) m
Thank you for your help.
Regards

Preparing a new version with a fix for this https://github.com/artberri/sidr/tree/reborn


Add monitoring of dependencies

To avoid manually checking for changed dependencies, now that we introduced version pinning (#305), maybe we could use https://requires.io to do that for us. As you can see in my fork the versions are extracted successfully.
There are three ways to get notified:

include a badge into our readme
email notifications
automatic pull request

As we only use those packages for testing my feeling is that the badge is sufficient and maybe emails to the track maintainers. It is free for public repos but it can only be activated by administrators.

I'd be happy to activate it. Ill take a look at it today.

OK, I think that should work.

Thank you.


Update needed.

This page requires update, there is unsupported Guest OS versions listed. N-1/N+1 its not honored which causes confusion. The mechanism to update Guest OS through Windows Update does not work properly for 2012R2 HOST hosting 2016/2012R2/2012

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

ID: cf5eedca-3535-d00e-98ab-9e25496a30a7
Version Independent ID: 797c37a7-ccd5-ed59-2dc3-5cf3892bbf3b
Content: Manage Hyper-V Integration Services
Content Source: WindowsServerDocs/virtualization/hyper-v/manage/Manage-Hyper-V-integration-services.md
Service: na
Product: windows-server
Technology: compute-hyper-v
GitHub Login: @KBDAzure
Microsoft Alias: kathydav

Hi @carlosmayol - please could you provide a bit more information? I can remove the unsupported guest OS versions, but not sure what you're referring to for N-1/+1 and Windows Update not working properly for 2012R2 Host

#assign:dknappettmsft

Hello, that has almost 2 years ago, happy someone is picking them at the end. Please just update OS versions. W11, WS2022 and check all out of support OSs

Thanks. I have tidied up this article, those version tables weren't hugely relevant anymore as integration services are updated through Windows Update or Linux kernel updates. There is a separate article already covering the supported guest OS versions in Hyper-V.  The updated article should be visible later today.
#please-close


Add support for remote temp file location for queued chunk reading

Draft for preliminary comments and review, still needing tests and a couple safety checks.
Mark the following tasks as done:

[x] Checked the codebase to ensure that your feature doesn't already exist.
[x] Checked the pull requests to ensure that another person hasn't already submitted the feature or fix.
[ ] Adjusted the Documentation.
[ ] Added tests to ensure against regression.

Description of the Change
This adds support for a new configuration key remote_temp_disk that, when not null, will store and retrieve spreadsheet chunks for queued reading at the specified filesystem disk in addition to the configured temp_path on the local server.
Why Should This Be Added?
Currently queued chunk reading is impossible on a multi-server setup, because it cannot be guaranteed that the spreadsheet chunk, a temporary file, will be uploaded and stored on the same server as the queue worker. In this case the ReadChunk job throws InvalidArgumentException: File "/tmp/xxx" does not exist.
Benefits

Possible Drawbacks

Verification Process

Applicable Issues
This is similar to, but not the same as, #1839, #1697, and #1827.

Hey @cmorbitzer thanks for the PR. I was thinking about doing something similar. I'm not really a fan of having multiple ways of dealing with temp files, so I'll probably merge that into 1; moving the normal temp file handling to the filesystem as well (default disk). Main goal is to not have a performance penalty when not using a clustered queue setup - as getting the file from S3 does slow down the process a bit. I'll use your PR as a base for the things I still want to change. Thanks for the contribution!

Thanks for the feedback! It's well received. I'll make the changes today for another review, as we have a release blocked by this.

Thanks @cmorbitzer !

@patrickbrouwers I addressed your comments as best as possible. Because PhpSpreadsheet requires files to be on the local filesystem, there was no way to consolidate both settings into one. There always has to be a local temp path even if we also need a separate (potentially remote) temp location.
The FilePathHelper has been substantially refactored to handle just about everything regarding temp files. This DRYs a lot of the existing code.
The remote_temp_disk will only be used for queued imports/exports because otherwise we know that the same server will be handling the entire job. You had said you didn't want to have two ways of handling temp files, but it seemed unnecessary to me to always be uploading and downloading temp files from a potentially remote location when we know it's just not necessary and would be a waste of resources. By having the FilePathHelper handle most everything, I hoped to make this as easy to understand and maintainable as possible. So in this case, since your desire for maintainability and performance were in conflict, I chose to side with performance.
This also includes some various minor refactors I noticed along the way. Feel free to revert those if I;m lacking institutional knowledge as to why it was the way it was.
I did not test this in any way yet, or give it a second look myself. I'm going to step away for a few hours before beginning that process. I wanted to commit this in case you or anyone else had comments in the meantime.

hey @cmorbitzer it looks fine to me. If you can make sure everything works in both queued as non-queued situations, than I'm happy to polish the last details myself after merging tomorrow.


Printing multiple copies on Plotter HP Designjet T520

Hello,
I am trying to print multiple copies to a plotter setup in CUPS. Plotter model is HP Designjet T520, since CUPS doesn't have driver for this model i am using an alternative driver HP Designjet T1100 pcl.
I am using library : https://github.com/williamkapke/ipp to send prints, Print request consists of :

data (image)
operation-attributes-tag {
job-name='any job name'
document-format = 'image/png' }
Job-attributes-tag {
copies : 2,
media : 'om_a-4-transverse_297.04x209.9mm'
}

This prints only 1 image from plotter.
As per Michael Sweet suggestion I also printed image from CUPS using command prompt :
lp -d UAStore10 -n 3 -o media=A4.Transverse PlotterA4.png
Same result only 1 print.
CUPS Version : 2.1.3
OS : Ubuntu 16.04.4 LTS
Could you please have a look at this one and suggest a solution ?
I already added this issue in CUPS github : https://github.com/apple/cups/issues/5352
Please let me know if I should do some more tests or any info is needed.
Thank you,
Sonia

There have been several fixes in cups-filters. Could you try with a newer version (or with a newer Ubuntu, like 18.04 LTS)?

since which version?

No  reaction of original poster any more, closing ...


Cannot add validation data if the validation data is loaded from a file.

Environment info
Operating System: Ubuntu 18.04
CPU/GPU model:
C++/Python/R version: Python 3.7
LightGBM version or commit hash: 2.3.0
Error message
[LightGBM] [Fatal] Cannot add validation data, since it has different bin mappers with training data
Traceback (most recent call last):
  File "bug.py", line 34, in <module>
    model = lgb.train(params, train_set=d_train, valid_sets=d_valid, early_stopping_rounds=100)
  File "/home/knakamura/.pyenv/versions/ashrae/lib/python3.7/site-packages/lightgbm/engine.py", line 232, in train
    booster.add_valid(valid_set, name_valid_set)
  File "/home/knakamura/.pyenv/versions/ashrae/lib/python3.7/site-packages/lightgbm/basic.py", line 1845, in add_valid
    data.construct().handle))
  File "/home/knakamura/.pyenv/versions/ashrae/lib/python3.7/site-packages/lightgbm/basic.py", line 47, in _safe_call
    raise LightGBMError(decode_string(_LIB.LGBM_GetLastError()))
lightgbm.basic.LightGBMError: Cannot add validation data, since it has different bin mappers with training data

Reproducible examples
The following one works:
d_train = lgb.Dataset(data=X_train, label=y_train)
d_valid = lgb.Dataset(data=X_valid, label=y_valid, reference=d_train)

# d_train.save_binary("train.bin")
# d_valid.save_binary("valid.bin")
#
# d_train = lgb.Dataset("train.bin")
# d_valid = lgb.Dataset("valid.bin", reference=d_train)

model = lgb.train(params, train_set=d_train, valid_sets=d_valid, early_stopping_rounds=100)

But the following one doesn't work:
d_train = lgb.Dataset(data=X_train, label=y_train)
d_valid = lgb.Dataset(data=X_valid, label=y_valid, reference=d_train)

d_train.save_binary("train.bin")
d_valid.save_binary("valid.bin")

d_train = lgb.Dataset("train.bin")
d_valid = lgb.Dataset("valid.bin", reference=d_train)

model = lgb.train(params, train_set=d_train, valid_sets=d_valid, early_stopping_rounds=100)

It worked. I don't know why I failed a lot of times. :disappointed:

I'm currently having the same problem with a similar hardward/software configuration. Any clue why is this happening?


Cannot install pynini==2.1.5 via pip

Hi, there. Thank you for your great job.
I am encountering an error even though Cython is installed. I use Linux (x86-64). The error message is:
ModuleNotFoundError: No module named 'Cython'
How do I solve this problem?

As is documented here, this was an issue with the v2.1.5 release. There is a release 2.1.5.post2 if you really need the 2.1.5 feature set. We are currently on version 2.1.6, though, and you probably want that one instead.
If you're on x86-64 Linux though why bother compiling from scratch? We ship a precompiled version for that platform. Run pip install --only-binary :all: pynini==2.1.5.post1 if you need 2.1.5 features or pip install --only-binary :all: pynini if you just want the newest version. Or, if you use Conda you can install these like conda install -c conda-forge pynini etc.


collapses after the trainval_net.py finishing its first  training iter

I am training my own dataset using fpn, i set the num_workers > 0 like =4, and then it collapses after the first training iter, but it will be ok if num_workers = 0, my device is following：
OS: Ubuntu 16.04
PyTorch version: pytorch 0.3.1
Python version: python 2.7
CUDA version: 8.0
GPU models : Tesla P40
[session 3][epoch  1][iter    0] loss: 4.3830, lr: 1.00e-03
			fg/bg=(115/397), time cost: 5.939724
			rpn_cls: 0.6945, rpn_box: 1.2795, rcnn_cls: 2.3964, rcnn_box 0.0127
Traceback (most recent call last):
  File "trainval_net.py", line 339, in <module>
    data = data_iter.next()
  File "/root/anaconda2/lib/python2.7/site-packages/torch/utils/data/dataloader.py", line 204, in __next__
    idx, batch = self.data_queue.get()
  File "/root/anaconda2/lib/python2.7/multiprocessing/queues.py", line 378, in get
    return recv()
  File "/root/anaconda2/lib/python2.7/site-packages/torch/multiprocessing/queue.py", line 22, in recv
    return pickle.loads(buf)
  File "/root/anaconda2/lib/python2.7/pickle.py", line 1388, in loads
    return Unpickler(file).load()
  File "/root/anaconda2/lib/python2.7/pickle.py", line 864, in load
    dispatch[key](self)
  File "/root/anaconda2/lib/python2.7/pickle.py", line 1139, in load_reduce
    value = func(*args)
  File "/root/anaconda2/lib/python2.7/site-packages/torch/multiprocessing/reductions.py", line 68, in rebuild_storage_fd
    fd = multiprocessing.reduction.rebuild_handle(df)
  File "/root/anaconda2/lib/python2.7/multiprocessing/reduction.py", line 155, in rebuild_handle
    conn = Client(address, authkey=current_process().authkey)
  File "/root/anaconda2/lib/python2.7/multiprocessing/connection.py", line 169, in Client
    c = SocketClient(address)
  File "/root/anaconda2/lib/python2.7/multiprocessing/connection.py", line 308, in SocketClient
    s.connect(address)
  File "/root/anaconda2/lib/python2.7/socket.py", line 228, in meth
    return getattr(self._sock,name)(*args)
socket.error: [Errno 111] Connection refused

i saw the same problem https://github.com/pytorch/pytorch/issues/1355, but his python verison is 3.x, and i tried that solution,  it does not works. @jwyang

thanks! actually, i have solved it by set cv2.setNumThreads(0).
anyway, thanks for your response.


Reshaping Feature Array

From OpenCV 1 to OpenCV 2/3 there is an additional array wrapping each of the feature points. So in OpenCV 1 two feature points of [20, 40] and [30,50] would have been returned as
[
     [20,40],
     [30, 50]
]

With the switch to numpy they were being returned as
[
     [
         [20,40]
     ],
     [
         [30, 50]
     ]
]

Added tests to verify the values of the features/facial points to catch this in the future.

Looks good, thanks again!


Introducing HotChocolate.Types.Scalars

By using scalar types for leaf fields, you can build expressive and easy to understand schemas.
the Guild implemented a very useful set of scalars in JavaScript. We want to port these scalars to HotChocolate and make them available for you.
We already unsealed all scalar types and created a test base that helps us test new scalars.
We have a many people in the community that would like to contribute to HotChocolate but do not know where to start. This issue is an incredibly good starter issue.
The Scalars

[x] scalar Time
[x] scalar DateTime
[ ] scalar Duration
[ ] scalar UtcOffset
[ ] scalar LocalDate
[ ] scalar LocalTime
[ ] scalar LocalEndTime
[x] scalar EmailAddress
[x] scalar NegativeFloat
[x] scalar NegativeInt Issues #2939 PR #2940
[x]  scalar NonEmptyString
[x] scalar NonNegativeFloatIssues #3013 PR #3015
[x] scalar NonNegativeInt
[x] scalar NonPositiveFloat
[x]  scalar NonPositiveInt
[x] scalar PhoneNumber
[x] scalar PositiveFloat Issues #2928 PR #2929
[x] scalar PositiveInt
[x]  scalar PostalCode Issue #3034
[x] scalar UnsignedFloat Will not be included
[x]  scalar UnsignedInt Issue #3036
[x]  scalar URL
[x] scalar ObjectID Belongs into the mongodb package
[x] scalar BigInt in the works
[x] scalar Long
[ ] scalar SafeInt
[x]  scalar UUID
[x] scalar GUID
[x] scalar HexColorCode
[x] scalar HSL
[ ]  scalar HSLA in the works
[ ] scalar IPv4 in the works
[ ] scalar IPv6
[ ] scalar ISBN in the works
[ ] scalar MAC
[ ] scalar Port
[ ]  scalar RGB
[ ] scalar RGBA
[ ] scalar USCurrency
[ ] scalar Currency
[ ] scalar JSON
[ ] scalar JSONObject
[x] scalar Byte
[ ] scalar Void

How to Contribute
If you would like to help, this is whats need to be done:

Choose a scalar you would like to implement from the list above and create an issue for it.
Use this issue as a template for title and body: #2928

Create a branch based on  feat/scalar-types.
Our branch policy is [three letter shortcut of your name]/[what your branch does].
In my case (Pascal Senn) this would be pse and i will implement the PositiveInt scalar
pse/add-positive-in-scalar

git clone https://github.com/ChilliCream/hotchocolate.git
git checkout feat/scalars
git checkout -b pse/add-positive-int-scalar

You will need contribution rights to push your changes to the repository once you finished your work. Reach out to Michael on slack or add a comment on this issue that you need contribution rights on the repository, we will provide you with the correct permissions.
Alternatively you can also fork the project and make the changes in your fork. You can then create a pull request from your fork onto ChilliCream/HotChocolate. This way you do not need to wait for the permissions.

You have to implement the scalar, add tests and add documentation. In what order you do this is up to you. You can take this pull request as a reference for what has to be implemented: #2929

You must take five methods in consideration
        /// This method defines if the given runtimeValue is possibly of this type
        ///
        /// This method is executed in various places. E.g. the validation uses this
        /// method to validate if a value is correct
        protected override bool IsInstanceOfType(string runtimeValue)
        {
            ...
        }

        /// This method defines if the given value syntax is possibly of this type
        ///
        /// This method is executed in various places. E.g. the validation uses this
        /// method to validate if a input value is correct
        protected override bool IsInstanceOfType(StringValueNode valueSyntax)
        {
            ...
        }

        /// Parses a runtime value of this type into a GraphQL value syntax representation.
        ///
        /// <exception cref="SerializationException">
        /// Unable to parse the given <paramref name="resultValue"/>
        /// into a GraphQL value syntax representation of this type.
        /// </exception>
        protected override string ParseLiteral(StringValueNode valueSyntax)
        {
            ...
        }

        /// Parses a result value of this into a GraphQL value syntax representation.
        ///
        /// <exception cref="SerializationException">
        /// Unable to parse the given <paramref name="resultValue"/>
        /// into a GraphQL value syntax representation of this type.
        /// </exception>
        protected override StringValueNode ParseValue(string runtimeValue)
        {
            ...
        }

        /// Serializes a runtime value into a result value (this method is used e.g. for variables)
        /// Returns false if serialization failed
        public override bool TrySerialize(object? runtimeValue, out object? resultValue)
        {
        }

        /// Deserializes a runtime value into a result value (this method is used e.g. for variables)
        /// Returns false if deserialization failed
        public override bool TryDeserialize(object? resultValue, out object? runtimeValue)
        {
        }

Create a pull request targeting the branch feat/scalar-types.
Wait for a review, resolve the comments, wait for a merge, celebrate 🎉

In my project, I have the requirement to work with Local DateTime, UTC DateTime, and DateTimeOffset.
The current version of the DateTime scalar supports both DateTimeOffset and UTC Date Time.
I am intending to define a LocalDateTime scalar in my project and can contribute it when its done.
I think it could also be useful to have a DateTimeObject type, that will allow consumers to pick what parts of a date they are interested in without needing to explicitly define the values in the schema.
For TimeZones I intend to define a TimeZoneId scalar based on IANA values, and a TimeZoneObject type with some values for display names, abbreviations, DST flag, etc..

In my project, I have the requirement to work with Local DateTime, UTC DateTime, and DateTimeOffset.
The current version of the DateTime scalar supports both DateTimeOffset and UTC Date Time.
I am intending to define a LocalDateTime scalar in my project and can contribute it when its done.
I think it could also be useful to have a DateTimeObject type, that will allow consumers to pick what parts of a date they are interested in without needing to explicitly define the values in the schema.
For TimeZones I intend to define a TimeZoneId scalar based on IANA values, and a TimeZoneObject type with some values for display names, abbreviations, DST flag, etc..

@DorianGreen We just had the same challenge in our project and ended up using NodaTime's LocalDateTime and a custom scalar type to map to and from that.


Pagination component not displayed correctly in IE11

Since updating to latest of carbon-components, the Pagination component is displayed incorrectly. It looks like the main change is that the bx--pagination__left element used to have a bx--select child, but now has a bx--form-item child.
Screenshot of older code that displayed correctly:

Screenshot of new code that no longer displays correctly:

@ryan76 this is for the React version, correct? I don't see bx--form-item in this repo

React issue, but fix will be done in here. Thanks for catching that!


📖 add better record syntax, improve pretty printing

I got tired of the unwieldy notation for signatures and structures... But I ran out of steam before I could update the pretty printer. Can someone please take over this PR? @TOTBWF ?

[x] update the parser and tests
[x] update the pretty printer

Going to rebase this against main now.

I'd like to get this merged today, can we have a look and see if it is OK?


Use the contract-build crate instead of calling cargo contract

This code is written in an almost functional style which may not be to everyone's taste. I'm happy to either rewrite deploy and call to this style, or convert this one to the imperative style :)

closes #21


Can not make list for Chinese characters entered via input method on safari

My code (Vue component)
<template>

    <button @click="makeList">make list</button>

</template>

it works on chrome

but not on safari

So the issue seems to be that Safari is stripping the
 (or other block tag) around the content when it replaces the latin characters with the Chinese one. This means when we go to convert the block to a list, we can't find a block. Interestingly, it works fine if there's at least one other sibling block present. So if you hit Enter to create a new line and then tried, it works fine.
I'd say this is a Safari bug, and I don't really have a good solution right now, sorry.


Add ability to customize Sass preprocessor output file names

Currently, we default to <filename>.min.css. This should be an overridable option.

To stay consistent with how most Sass processors work, let's not permit an option to change the basename. However, we already have an option to change the suffix, which is configurable using the suffix config option.
http://grow.io/docs/preprocessors/#sass


Accessibility usage of the service manual

What
To measure the usage of the accessibility section of the service manual.
Why
To determine whether it still meets user needs given the changes in accessibility regulations.
Done when

[ ] Collect data on usage of the accessibility pages on the service manual
[ ] Analyse data collected
[ ] Share findings

Duplicated issue (see epic - Accessibility guidance (update + assess existing guidance) #669


Export not working for me

I don't know if you guys are still mantaining this project, but exporting in general and also, exporting to CSV doesn't work in lastest Chromium, Firefox and Edge. This is both in the demo and my local git clone.

Please let me know if you have a workaround for this issue, I'm from Argentina and I'm interested in using openCPQ.

Sorry for the mistake, I realized this should be in the openCPQ-example-optical-transport repo


Part of debug text originates from server?

I just noticed that seemingly the memory stats for the headless server rather than the connected client shows in the F3 debug stats. Showed 4 GB max memory which is what the server was launched with, but not what my client was running.
It makes sense that some of the debug text matches the server, like your coordinates. Other stuff like the timings surely match the client. We might want to take a closer look there :-)

On a not-really-related note but I'm not sure yet if it is just me and maybe it'll be encountered while testing this. I found that on some logins on a headless server the camera would be off vs my monkey head.
First I was looking at myself from above/behind my head so looking down meant I was looking through my head. Then on another login I seemingly disconnected entirely from my avatar, which stayed where I was when I quit / was disconnected as the server was stopped abruptly at that point. Another attempt had no weirdness at all and then I was behind my head again.

@Cervator I've been playing with this for the past few days off and on. And I don't think that it's a problem with the game reading the memory from the server, I think it is a problem with
gradlew game -D-XmxWHATEVER
not passing the values properly, or them getting dropped somewhere farther down the line...
What I've tried (RESULT answer is from the in-game F3 menu)
As a control, I just ran

./gradlew game

RESULT: max memory: 3559.5

Then for the actual tests:

./gradlew game -Xms512m -Xmx1024m

RESULT: max memory: 3559.5

./gradlew game -D-Xmx1024m

RESULT: max memory: 3559.5

./gradlew game -D-Xmx6144m

RESULT: max memory: 3559.5

...
More of the same...

Log output: (cat Terasology-init.log | grep Max.)
...
[main] INFO  o.terasology.engine.TerasologyEngine - Max. Memory: 3559 MiB
...
Here's why I think this:
I made a little test file that uses the exact same static method that both the "TerasologyEngine" and "DebugOverlay" is using; (Forgive the spacing, github apparently hates me :P)
public class test1 { 	public static void main(String[] args) { 		System.out.println(Runtime.getRuntime().maxMemory() / 1048576.0);       } }
Executed this file (After compile of course)

java test1

... And you guessed it :D .. The output is -->
3559.5
I then ran

java -Xmx6114m test1

... And the output was -->
5461.5
Which leads me to believe that these command line params for gradlew are getting lost somewhere.
Any thoughts on where I might look to fix this issue? I'm kindof lost on where to begin on this.
Side note: Getting really tired of seeing the number 3559.5    -_-

Just to repost some info here: after some IRC discussion and experimenting by @AWildBeard we found that changing the memory settings with a gradlew run may be oddly difficult (the game task has its own config), while IntelliJ's run configs respect the memory settings pretty well.
That may be something to keep in mind and investigate further at some point.

Addition: add NORTH, EAST ... directions to the debug text. See https://github.com/Terasology/Compass how how it could be done

In the scripts, I think the Runtime.getRuntime() function runs on the server, while time.getFPS() runs on client. I tested this by using GPU 0 and GPU 1, I got 60 fps on GPU 1 while 25 fps on GPU 0. And about the mem usage stats, I think it shows for the server because the max memory exceeds on my task manager for the game (max mem is 1365 MB but it gets over 1500 MB). So by doing these tests I believe FPS is client-side and mem stats is server-side.
I have modified the script "DebugOverlay.java" organizing what is server's and what's for client, including Memory Heap logic to read Memory stats correctly for client-side.
Also added Compass direction debug so we can see which direction we are heading.
Pull request submitting :)
Let me know if anything is wrong!


Update teams-privacy-oce-overview.md

Modified the URL Preview Service section by adding the Bing Content Validation Service that URLs might be sent to as well

Hello @MicrosoftHeidi, could you please review and sign off my change?

@MicrosoftHeidi @JohanFreelancer9 I don't see a merge option, most probably because I don't have write access to the repository. Can someone please merge the PR?

@MicrosoftHeidi @JohanFreelancer9 I don't see a merge option, most probably because I don't have write access to the repository. Can someone please merge the PR?

@scanum, could you please assist? Thanks.

@MicrosoftHeidi
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


ACTIVECOM-1606 Fix webpage link changed to https issue

JIRA

https://jirafnd.dev.activenetwork.com/browse/ACTIVECOM-1606?filter=34274

Test link:

http://a3coreint10.dev.activenetwork.com/running/articles/10-healthy-ish-ice-cream-recipes-for-runners

On it

@RogerTongTong Exactly, Just checked at A3 side. imageName =  'MediaGalleryButton' should be a good decision.

Look great 👍


Refactor package name to m3db and move top level interfaces to interfaces/m3db/

cc @xichen2020 @martin-mao @mmihic

LGTM

Coverage remained the same at 72.377% when pulling 6fb449ea2936b4fe47d20b7e3059acf14070129a on refactor-package-name into 25401e48ef270cf7f6dc3b0f1e48009b205f8569 on master.


Add debugging support?

Is proto-repl ever going to get support for debugging such as Cider: https://github.com/clojure-emacs/cider/blob/master/doc/debugging.md or Cursive?

I'd support the addition of a traditional debugger. I don't have plans to implement it currently though. My debugging needs are mostly met by the current features.
If someone were going to do this I would recommend taking a look at how the vs code plugin for Clojure does it.

On Apr 11, 2017, at 9:43 PM, Didier A<EMAIL_ADDRESS>wrote:
Is proto-repl ever going to get support for debugging such as Cider: https://github.com/clojure-emacs/cider/blob/master/doc/debugging.md or Cursive?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Update requirements.txt

Fixes deployment issues on Heroku with size
Added Postgres driver

@IonesioJunior kindly review.


typo in readme - install instructions

I believe the fourth line of the package install instructions ("cd Adafruit_Python_GPIO") should read "cd Adafruit_Python_BME280".

closing old issues. i think those instructions are correct, they're for the GPIO dependancies


42 key features cards

Loan Product Key-features-block
Fix #42
Test URLs:

Before: https://main--piramal--aemsites.hlx.page/home-loan
After: https://42-key-features-cards--piramal--aemsites.hlx.page/home-loan

@asthabh23 the alignment/font-size should be fine now


[No ticket] Don't keep generated code in repository

Short description if any.
Author Todo List:

[x] Add/adjust tests (if applicable)
[x] Build in CI passes
[x] Latest master revision is merged into the branch
[x] Self-Review
[x] Set Ready For Review status

We are not packaging them as far as I can tell. I wanted to keep them around to not break something subtle in our packaging steps.
But you are right, we can probably remove it entirely. updated


Some comments on commands and options

#adding the file or folder inside particular parent file(test) using its file id
$ drive add_remote --file mailer.py --pid 1RJOWpW5MuP9RXpgZbp9OdauhaBtJd49g

pid in command-line context more familiarly means "process id". I'd prefer parent-id.
The help text sounds like you can add a file inside a file, that doesn't seem to make sense, should be folder?

view-files: list your files; filter them by name, type.

Rather query or search. And then rename the cat command to view-file :wink:

add_remote: upload existing local file to drive and get it linked.

Just add, remote is implied. Also, "remote" here invites confusion with git remote.

status: list changes made to local files since last pull or pull.

Also shows files updated remotely?

@jean

pid in command-line context more familiarly means "process id". I'd prefer parent-id.

Makes sense but IMO it's a convention to check the help text ( for first timers atleast ) where it clearly states the usage, I that case i'd prefer a pid.

Also, quick note @nurdtechie98 :

  --pid TEXT   specify particular folder id/sharing_link of the folder under
               which remote must must be added

It'd be easier to understand if the help text is just - specify particular folder id/sharing_link under which remote must must be added.
@nurdtechie98 pinging you, here.
Let me know what you think of these.
Thanks

@jean Closing this for now.
Feel free to open in case you feel something is wrongly interpreted / addressed ;)

@inishchith I'm fine with your preference regarding pid, it isn't ambiguous in the context of this program.
I do think it's an easy-to-avoid false rhyme with usage elsewhere, and I think it's less explicit than it needs to be (parent-id is self-documenting).
I'm a bit disappointed you don't respond to the other suggestions:

view-files: list your files; filter them by name, type.
Rather query or search. And then rename the cat command to view-file wink

cat is a terrible name for "view-file". If ever there was a Unx fossil, that's one. I.e. you disregard the meaning of pid elsewhere (in all the process management tools like kill, ps, top, etc.), but for cat you rely on the reader's familiarity with a common usage of the old tool (where more or less is actually normally more appropriate). The expansion (concatenate) doesn't even make sense. I mean, I love the ancient nerd humour of the cryptic old Unx commands, but we're in different times now, we're all juggling myriads of new tools from all over the place, and clear boring naming is a big help.
view-files also does not imply any filtering. filter/query/search does.

add_remote: upload existing local file to drive and get it linked.
Just add, remote is implied. Also, "remote" here invites confusion with git remote.

I stand by that, I don't see that _remote adds anything.
Since there aren't any collisions, the old names can silently keep working for the sake of backwards compatibility.


docs: improve doc site

This changes the structure of the site to be more similar to the python version with a section called Model I/O and data connection. https://python.langchain.com/docs/. The lint timed out.
PR Checklist

[x] Read the Contributing documentation.
[x] Read the Code of conduct documentation.
[x] Name your Pull Request title clearly, concisely, and prefixed with the name of the primarily affected package you changed according to Good commit messages (such as memory: add interfaces for X, Y or util: add whizzbang helpers).
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. Fixes #123).
[x] Describes the source of new concepts.
[x] References existing implementations as appropriate.
[x] Contains test coverage for new functions.
[x] Passes all golangci-lint checks.

I think the examples should be organized in different sub directories, but I dont know how that would effect the build-examples ci and the makefile.


just fixing a typo

1. Why is this change necessary?
Because it fixes Typos
2. What does this change do, exactly?
It's just fixing a Typo
3. Describe each step to reproduce the issue or behaviour.
Try to delete a file and in case of an error. See the error message with a typo.
4. Please link to the relevant issues (if any).
No links available
5. Checklist

[❌] I have written tests and verified that they fail without my change
[✅] I have squashed any insignificant commits
[✅] I have created a changelog file with all necessary information about my changes
[❌] I have written or adjusted the documentation according to my changes
[❌] This change has comments for package types, values, functions, and non-obvious lines of code
[✅] I have read the contribution requirements and fulfil them.

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

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-19101
Please use this issue to track the state of your pull request.

Thanks for your contribution! :blue_heart: :tada:


fix: Return app_dict as django does to allow display a single app adm…

…in view
FIX ISSUE #170
Removed two lines of code that make Django launch an error because the string doesn't allow string indexes, to match more closely Django behavior when handling the model_dicts  I removed the two lines in this commit.

Hi, @MaistrenkoAnton , @Dvoretska , @pablop94  could you review if this is okay and merge it? I will release an app using this project to production and need this fix for the app to work... thanks beforehand for the great app you did here... I hope this PR actually doesn't break anything in another place, I watched how the Django framework does the same and removed the lines that seem to cause the reported error... have a nice day


DartPad export - server enabled

Export capabilities enabled - server located at http://export-api.dart-services.appspot.com.
@devoncarew @lukechurch

@lukechurch ready for review.

@Georgehe4 Please address these minor comments, then lgtm.
Please could you also update the title of the PR to be a little more explanatory - e.g. "Enable pad export"

Merging.


[Lecture13][1214] Regarding the Embedding Space in the VQ-VAE Pipeline

I have a question regarding the embedding space in the VQ-VAE pipeline. Is the embedding space used in the same form throughout the process, or is it updated during training?
Thank you for your time and help.
-20221186 Jihyeon Kim-

@kimjyan20221186
Thank you for the comment!
In the training phase, the embedding vectors (codebook) are updated. However, in its inference stage, the embeddings are fixed.

@yjyoo3312 I'm grateful for the detailed description of  embeddings change😊


Speech service timeout quickly and cut off dictation

Describe the bug
when using enable dictation mode, user pauses in their speech for a second or 2 which is natural, but the speech to text interprets this as being done. It should have a way to disable that so that you can just speek all text and punctuation then stop at the end.
To Reproduce
speak paragraphs with pauses where user thinks about next statement and speech recognition fires events and inserts punctuation automatically and thinks the user is finished/finishing. but they are just thinking about the next statement, or continuation of the same sentence and just thinking of what they want to say
Expected behavior
be able to just tell speech recog to not have silent detection for a timeout.
Version of the Cognitive Services Speech SDK
1.12
Platform, Operating System, and Programming Language
windows, C# project, winforms.
Additional context
no errors logged.

Please see https://stackoverflow.microsoft.com/questions/183670/ for information on configurability of the silence timeouts used by the service. Unfortunately there is currently no way to configure or disable the timeout after which a phrase is detected and a final result is generated.

Just to update, I've added a work item to our backlog, to make the timeout for phrase detection configurable if possible (it has a service dependency). We will consider it for a future release and update this issue once the work is done.

@pankopon Hi ... the stack overflow link is internal. We also want to configure the VAD to be a little more patient. We have played around different variables but it doesn't work properly for the end of speech. Here are a couple:
PropertyId.SpeechServiceResponse_RecognitionLatencyMs
PropertyId.SpeechServiceConnection_EndSilenceTimeoutMs
PropertyId.SpeechServiceConnection_InitialSilenceTimeoutMs

we tried them all but no luck.

Was the ability to customize the silence timeout ever implemented? I haven’t been able to find it in the documentation. I created a prototype using this service and noticed this issue right away. It’s very unintuitive compared to other solutions, like Apple’s speech recognition framework, but the accuracy and performance are excellent otherwise!

Yes, you can customize the end of phrase silence timeout with the Segmentation Timeout property.
There's a writeup on silence here:
https://learn.microsoft.com/azure/cognitive-services/speech-service/how-to-recognize-speech?pivots=programming-language-csharp#change-how-silence-is-handled

Thank you! I’m surprised I didn’t come across that article. I did figure out how to change the silence timeout earlier this morning and things are working as expected now.


Fix stop joining thread before interrupting Selector.select

Fixes #259

Looks like a good fix of the main cross-platform bug to me. Since #259 involves Android but does not specify being post-Honeycomb, should we also include the Selector.wakeup() call before considering it solved?

I'm not an Android developer so I can't really test the change, but here you go.
Confirmed to work on jdk8.

:+1:


cudaMalloc failed: out of memory

Hello. When I run the demo.py, I meet the problem below. My workstation is configured with an NVIDIA GTX 1070(8G). The version of Mxnet is 1.0.0. Looking forward to your reply.  If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPE to NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.
Stack trace returned 8 entries:
[bt] (0) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x28965c) [0x7fef5294865c]
[bt] (1) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x24511c4) [0x7fef54b101c4]
[bt] (2) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x2455d5b) [0x7fef54b14d5b]
[bt] (3) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x2455f36) [0x7fef54b14f36]
[bt] (4) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x245275b) [0x7fef54b1175b]
[bt] (5) /home/smiles/anaconda2/bin/../lib/libstdc++.so.6(+0xb7260) [0x7fefc3111260]
[bt] (6) /lib/x86_64-linux-gnu/libpthread.so.0(+0x8184) [0x7fefc8747184]
[bt] (7) /lib/x86_64-linux-gnu/libc.so.6(clone+0x6d) [0x7fefc7d66ffd]
terminate called after throwing an instance of 'dmlc::Error'
what():  [19:45:05] src/engine/./threaded_engine.h:359: [19:45:05] src/storage/./pooled_storage_manager.h:107: cudaMalloc failed: out of memory
Stack trace returned 10 entries:
[bt] (0) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x28965c) [0x7fef5294865c]
[bt] (1) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x29c846f) [0x7fef5508746f]
[bt] (2) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x29ca628) [0x7fef55089628]
[bt] (3) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x29ceb41) [0x7fef5508db41]
[bt] (4) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x2a4ec11) [0x7fef5510dc11]
[bt] (5) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x3b6879b) [0x7fef5622779b]
[bt] (6) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x268f834) [0x7fef54d4e834]
[bt] (7) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x246960b) [0x7fef54b2860b]
[bt] (8) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x246e760) [0x7fef54b2d760]
[bt] (9) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x2450f1d) [0x7fef54b0ff1d]
A fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPE to NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.
Stack trace returned 8 entries:
[bt] (0) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x28965c) [0x7fef5294865c]
[bt] (1) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x24511c4) [0x7fef54b101c4]
[bt] (2) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x2455d5b) [0x7fef54b14d5b]
[bt] (3) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x2455f36) [0x7fef54b14f36]
[bt] (4) /home/smiles/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x245275b) [0x7fef54b1175b]
[bt] (5) /home/smiles/anaconda2/bin/../lib/libstdc++.so.6(+0xb7260) [0x7fefc3111260]
[bt] (6) /lib/x86_64-linux-gnu/libpthread.so.0(+0x8184) [0x7fefc8747184]
[bt] (7) /lib/x86_64-linux-gnu/libc.so.6(clone+0x6d) [0x7fefc7d66ffd]
Aborted (core dumped)

Maybe GPU memory is not enough. You can try to change KEY_FRAME_INTERVAL= below 5.
I use   KEY_FRAME_INTERVAL=5 is ok.
Hope to useful for you!

Hi @Feywell , I am having the same problem...how did you change the key frame interval?
Thanks.

Actually, it is ok when I train the code. But test phase, the error is throw out. I change the KEY_FRAME_INTERVAL in config.py and min_diff and max_diff. Hope it is helpful for you.

@Feywell Thank you for your reply. I faced that error when I was running the demo. My GPU has 8 GB of memory so I am not sure why would it give this error. I changed the KEY_FRAME_INTERVAL to 5 instead of 9 and now I do not see that error anymore...however I am getting a new error Check failed: shape_assign(&(*in_shape)[i], dshape) Incompatible input shape: expected (11,0,600,1000), got (19,3,600,1000)
:( not sure yet what it means

@scharifa You must encountered this error in test phase.
If you run demo.py, you should change demo.py like follows:

max_data_shape = [[('data', (1, 3, max([v[0] for v in cfg.SCALES]), max([v[1] for v in cfg.SCALES]))),
                   ('data_cache', (11, 3, max([v[0] for v in cfg.SCALES]), max([v[1] for v in cfg.SCALES]))),
                   ('feat_cache', ((11, cfg.network.FGFA_FEAT_DIM,
                                            np.ceil(max([v[0] for v in cfg.SCALES]) / feat_stride).astype(np.int),
                                            np.ceil(max([v[1] for v in cfg.SCALES]) / feat_stride).astype(np.int))))]]

If you run 'fgfa_rfcn_end2end_train_test.py',you should change function  get_init_batch in core/loader.py  as follow:

extend_data = [{'data': data[0]['data'] ,
'im_info': data[0]['im_info'],
'data_cache': np.zeros((7, 3, max([v[0] for v in self.cfg.SCALES]), max([v[1] for v in self.cfg.SCALES]))),
'feat_cache': np.zeros((7, self.cfg.network.FGFA_FEAT_DIM,
np.ceil(max([v[0] for v in self.cfg.SCALES]) / feat_stride).astype(np.int),
np.ceil(max([v[1] for v in self.cfg.SCALES]) / feat_stride).astype(np.int)))}]

@Feywell Thank you so much. That really helps.
I just had to change it to
max_data_shape = [[('data', (1, 3, max([v[0] for v in cfg.SCALES]), max([v[1] for v in cfg.SCALES]))),
('data_cache', (9, 3, max([v[0] for v in cfg.SCALES]), max([v[1] for v in cfg.SCALES]))),
('feat_cache', ((9, cfg.network.FGFA_FEAT_DIM,
np.ceil(max([v[0] for v in cfg.SCALES]) / feat_stride).astype(np.int),
np.ceil(max([v[1] for v in cfg.SCALES]) / feat_stride).astype(np.int))))]]


TileSet error spam when opening a scene with (old) TileMap

Godot version:
3.2 master (34224d5)
OS/device including version:
Any
Issue description:
When I opened the Node2D.tscn scene in the attached project, I get this errors repeatedly:
ERROR: tile_get_shape_transform: Index p_shape_id=0 out of size (tile_map[p_id].shapes_data.size()=0)
   At: scene/resources/tile_set.cpp:753.
ERROR: tile_get_shape_transform: Index p_shape_id=0 out of size (tile_map[p_id].shapes_data.size()=0)
   At: scene/resources/tile_set.cpp:753.
ERROR: tile_get_shape: Index p_shape_id=0 out of size (tile_map[p_id].shapes_data.size()=0)
   At: scene/resources/tile_set.cpp:734.
ERROR: tile_get_shape_one_way: Index p_shape_id=0 out of size (tile_map[p_id].shapes_data.size()=0)
   At: scene/resources/tile_set.cpp:782.
ERROR: tile_get_shape_one_way_margin: Index p_shape_id=0 out of size (tile_map[p_id].shapes_data.size()=0)
   At: scene/resources/tile_set.cpp:801.

The TileSet was generated from a scene in Godot 3.1, and it does not produce errors there.
Steps to reproduce:
See above.
Minimal reproduction project:
jetpaca-tilemap.zip

This is caused by this commit - https://github.com/godotengine/godot/commit/02319dceb2e17184eb765c67719a306f56dafc1b.
Before this commit, errors were simply silenced.

Had a quick look, I think some of those changes should likely be undone. The whole class seems to be written with the assumption that you can call things like tile_get_shape_transform(int p_id, int p_shape_id) and get Transform2D() back if the p_id or p_shape_id is invalid.
So either we keep allowing this (that's bad design, but the whole class seems designed around it), or we would have to add stuff like this in many places:
diff --git a/scene/resources/tile_set.cpp b/scene/resources/tile_set.cpp
index f577a2e14..4d20413b2 100644
--- a/scene/resources/tile_set.cpp
+++ b/scene/resources/tile_set.cpp
@@ -756,7 +756,11 @@ Transform2D TileSet::tile_get_shape_transform(int p_id, int p_shape_id) const {
 }

 void TileSet::tile_set_shape_offset(int p_id, int p_shape_id, const Vector2 &p_offset) {
-	Transform2D transform = tile_get_shape_transform(p_id, p_shape_id);
+
+	Transform2D transform;
+	if (tile_map.has(p_id) && p_shape_id < tile_map[p_id].shapes_data.size()) {
+		transform = tile_get_shape_transform(p_id, p_shape_id);
+	}
 	transform.set_origin(p_offset);
 	tile_set_shape_transform(p_id, p_shape_id, transform);
 }

Above snippet can be a bit prettier using the proper methods for that though:
if (has_tile(p_id) && p_shape_id < tile_get_shape_count(p_id))


⚠ ClamAV has degraded performance

In 66d8d90, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 62 ms

Resolved: ClamAV performance has improved in 7b427c1.


Disable reset password for Facebook users

Hi.
It looks like there's nothing that I can tweak in order to disable sending reset emails to Facebook users.
Is that true?
Thank you

Related stackoverflow question: https://stackoverflow.com/questions/28940325/rails-devise-disable-password-recovery-for-certain-user-types


Cannot import into react

Is there a specific method I need to use to use this in react?
I use any of the examples like so
const sma = require('technicalindicators').sma; var prices = [1,2,3,4,5,6,7,8,9,10,12,13,15]; var period = 10; sma({period : period, values : prices})
I get this error:
<EMAIL_ADDRESS>Attempted import error: 'deprecationWarn' is not exported from '@tensorflow/tfjs-core'.

Same here !

Tensorflow removed as part of 3.0


Intersection order matters when inferring generics.

Bug Report

🔎 Search Terms
intersection order
inferring generics

🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about inferring generics.
⏯ Playground Link

Playground link with relevant code
💻 Code

type Foo<A, B> = (p: { foo: A }) => { foo: B };
type Bar<A, B> = (p: { bar: A }) => { bar: B };

type FooBar<A, B> = Foo<A, B> & Bar<A, B>;
// type FooBar<A, B> = Bar<A, B> & Foo<A, B>;		// If `FooBar` is changed to this, the test results reverse.

type Value<T> = { value: T };

declare function usingFoo<A, B>(p1: Value<A>, p2: Foo<A, B>): Value<B>;
declare function usingBar<A, B>(p1: Value<A>, p2: Bar<A, B>): Value<B>;

declare const value: Value<"A">;
declare const foobar: FooBar<"A", "B">;

const test1 = usingFoo(value, foobar); // is: Value<unknown>     - is undesired, desired is: Value<"B">
const test2 = usingBar(value, foobar); // is: Value<"B">         - is desired

🙁 Actual behavior
TypeScript can only infer the return type properly depending on the order of the elements in the FooBar intersection.
🙂 Expected behavior
The order of the elements shouldn't matter, TypeScript should be able to infer the type of both test1 and test2 correctly.

Duplicate of #26591 by way of #50975/#50708.


What is the difference between clingo and clasp projects

clasp only installs bin/clasp, and clingo installs more files including bin/clasp.
We have both clasp and clingo as separate ports on FreeBSD.
Is clasp entirely included in clingo and it doesn't make sense to install it separately?

clasp and clingo are bundled together. Since many years, clingo releases immediately follow clasp releases. In principle, it is not necessary to have two different packages. It is possible to disable installation of clasp's binaries via an option when installing clingo. I can imagine that this leads to a smaller installation if someone only needs clasp. But we are talking about one or two megabytes here.

Ok, thanks.


Require engine automatically

Summary
Currently anyone using the framework has to require view_component/engine manually, usually by putting gem "view_component", require: "view_component/engine" in the Gemfile or requiring it specifically in application.rb. Forgetting to do so can lead to weird reloading problems and other subtle issues as evidenced in #560.
This PR adds the relevant require to avoid these issues.

That's a great improvement for newcomers! :clap: @camertron

@camertron @joelhawksley Hi both, a massive thank you for this change 🙌 Just a quick heads up that I bumped into an issue which is that it seems that merging this PR updated the docs i.e it removed the , require: "view_component/engine",  however the corresponding change hasn't yet made it into the latest version. So when I ran bundle install and rendered my first component, I was confused as to why it wasn't then updating.
Not sure I have a suggesting for how situations like that can be mitigated, but could potentially be confusing for newcomers in the gap between PRs being merged and a new version being released.
Was livestreaming when I bumped into this, so if you'd like to see me writing this comment (which is super meta), you can do here.

@maxshelley I guess we should either add a disclaimer to the docs or release a new version now (or both?)
@camertron @joelhawksley by the way, what about adding an explicit and more visible mention at the top of the changelog that the engine require can be removed from the Gemfile after upgrading?

@maxshelley thanks for flagging this. I think there is definitely some room for us to improve our release process.
In the meantime, I've published v2.43.0 that includes the change from this PR. (It turns out we hadn't shipped the release notes for v2.42.0 either 😬)

@maxshelley oops that's my bad, I totally thought we had released this in 2.42 😅


Deploy Example Notebooks

These are the initial examples of deployment with BentoML and Ray.

The tests are failing because the converter used to make script files out of notebooks are interpreting writefile cells as code. These have duplicate code present in previous cells and imports in odd places.

Hm tricky. The code goes through jupytext to convert from notebook to script, and then gets piped into checkers. The script that gets inputted to the checks is below. Perhaps I can use sed to remove everything between # %%writefile and the end of the cell # -.
Not sure what to do for the autofix, which will convert notebook->script->do the fixes and then -> back to notebook. Which means I couldn't delete it.
As an easy alternative, you could avoid writing the file mednist_classifier_start.py on the fly? And simply tell people to load it up and have a look?
# ---
# jupyter:
#   jupytext:
#     text_representation:
#       extension: .py
#       format_name: light
#       format_version: '1.5'
#       jupytext_version: 1.9.1
#   kernelspec:
#     display_name: Python [conda env:monai]
#     language: python
#     name: conda-env-monai-py
# ---

# # Deploying a MedNIST Classifier with BentoML
#
# This notebook demos the process of packaging up a trained model using BentoML into an artifact which can be run as a local program performing inference, a web service doing the same, and a Docker containerized web service. BentoML provides various ways of deploying models with existing platforms like AWS or Azure but we'll focus on local deployment here since researchers are more likely to do this. This tutorial will train a MedNIST classifier like the [MONAI tutorial here](https://github.com/Project-MONAI/tutorials/blob/master/2d_classification/mednist_tutorial.ipynb) and then do the packaging as described in this [BentoML tutorial](https://github.com/bentoml/gallery/blob/master/pytorch/fashion-mnist/pytorch-fashion-mnist.ipynb).

# ## Setup environment

# +
# !python -c "import monai" || pip install -q "monai[pillow, tqdm]"
# !python -c "import matplotlib" || pip install -q matplotlib
# !python -c "import bentoml" || pip install -q bentoml

# %matplotlib inline
# -

# ## Setup imports

# + tags=[]
# Copyright 2020 MONAI Consortium
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#     http://www.apache.org/licenses/LICENSE-2.0
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.

import os
import shutil
import tempfile
import glob
import matplotlib.pyplot as plt
import PIL.Image
import torch
import numpy as np

from ignite.engine import Events

from monai.apps import download_and_extract
from monai.config import print_config
from monai.networks.nets import densenet121
from monai.engines import SupervisedTrainer
from monai.transforms import (
    AddChannel,
    Compose,
    LoadImage,
    RandFlip,
    RandRotate,
    RandZoom,
    ScaleIntensity,
    ToTensor,
)
from monai.utils import set_determinism

set_determinism(seed=0)

print_config()
# -

# ## Download dataset
#
# The MedNIST dataset was gathered from several sets from [TCIA](https://wiki.cancerimagingarchive.net/display/Public/Data+Usage+Policies+and+Restrictions),
# [the RSNA Bone Age Challenge](http://rsnachallenges.cloudapp.net/competitions/4),
# and [the NIH Chest X-ray dataset](https://cloud.google.com/healthcare/docs/resources/public-datasets/nih-chest).
#
# The dataset is kindly made available by [Dr. Bradley J. Erickson M.D., Ph.D.](https://www.mayo.edu/research/labs/radiology-informatics/overview) (Department of Radiology, Mayo Clinic)
# under the Creative Commons [CC BY-SA 4.0 license](https://creativecommons.org/licenses/by-sa/4.0/).
#
# If you use the MedNIST dataset, please acknowledge the source.

# + tags=[]
directory = os.environ.get("MONAI_DATA_DIRECTORY")
root_dir = tempfile.mkdtemp() if directory is None else directory
print(root_dir)

resource = "https://www.dropbox.com/s/5wwskxctvcxiuea/MedNIST.tar.gz?dl=1"
md5 = "0bc7306e7427e00ad1c5526a6677552d"

compressed_file = os.path.join(root_dir, "MedNIST.tar.gz")
data_dir = os.path.join(root_dir, "MedNIST")
if not os.path.exists(data_dir):
    download_and_extract(resource, compressed_file, root_dir, md5)

# +
subdirs = sorted(glob.glob(f"{data_dir}/*/"))

class_names = [os.path.basename(sd[:-1]) for sd in subdirs]
image_files = [glob.glob(f"{sb}/*") for sb in subdirs]

image_files_list = sum(image_files, [])
image_class = sum([[i] * len(f) for i, f in enumerate(image_files)], [])
image_width, image_height = PIL.Image.open(image_files_list[0]).size

print(f"Label names: {class_names}")
print(f"Label counts: {list(map(len, image_files))}")
print(f"Total image count: {len(image_class)}")
print(f"Image dimensions: {image_width} x {image_height}")
# -

# ## Setup and Train
#
# Here we'll create a transform sequence and train the network, omitting validation and testing since we know this does indeed work and it's not needed here:

train_transforms = Compose(
    [
        LoadImage(image_only=True),
        AddChannel(),
        ScaleIntensity(),
        RandRotate(range_x=np.pi / 12, prob=0.5, keep_size=True),
        RandFlip(spatial_axis=0, prob=0.5),
        RandZoom(min_zoom=0.9, max_zoom=1.1, prob=0.5),
        ToTensor(),
    ]
)

# +
class MedNISTDataset(torch.utils.data.Dataset):
    def __init__(self, image_files, labels, transforms):
        self.image_files = image_files
        self.labels = labels
        self.transforms = transforms

    def __len__(self):
        return len(self.image_files)

    def __getitem__(self, index):
        return self.transforms(self.image_files[index]), self.labels[index]

# just one dataset and loader, we won't bother with validation or testing
train_ds = MedNISTDataset(image_files_list, image_class, train_transforms)
train_loader = torch.utils.data.DataLoader(train_ds, batch_size=300, shuffle=True, num_workers=10)
# -

device = torch.device("cuda:0")
net = densenet121(spatial_dims=2, in_channels=1, out_channels=len(class_names)).to(device)
loss_function = torch.nn.CrossEntropyLoss()
optimizer = torch.optim.Adam(net.parameters(), 1e-5)
max_epochs = 4

# +
def _prepare_batch(batch, device, non_blocking):
    return tuple(b.to(device) for b in batch)

trainer = SupervisedTrainer(device, 5, train_loader, net, optimizer, loss_function, prepare_batch=_prepare_batch)

@trainer.on(Events.EPOCH_COMPLETED)
def _print_loss(engine):
    print(f"Epoch {engine.state.epoch}/{engine.state.max_epochs} Loss: {engine.state.output['loss']}")

trainer.run()
# -

# The network will be saved out here as a Torchscript object but this isn't necessary as we'll see later.

torch.jit.script(net).save("classifier.zip")

# ## BentoML Setup
#
# BentoML provides it's platform through an API to wrap service requests as method calls. This is obviously similar to how Flask works (which is one of the underlying technologies used here), but on top of this is provided various facilities for storing the network (artifacts), handling the IO component of requests, and caching data. What we need to provide is a script file to represent the services we want, BentoML will take this with the artifacts we provide and store this in a separate location which can be run locally as well as uploaded to a server (sort of like Docker registries).
#
# The script below will create our API which includes MONAI code. The transform sequence needs a special read Transform to turn a data stream into an image, but otherwise the code like what was used above for training. The network is stored as an artifact which in practice is the stored weights in the BentoML bundle. This is loaded at runtime automatically, but instead we could load the Torchscript model instead if we wanted to, in particular if we wanted an API that didn't rely on MONAI code.
#
# The script needs to be written out to a file first:

# +
# %%writefile mednist_classifier_bentoml.py

from typing import BinaryIO, List
import numpy as np
from PIL import Image
import torch

import monai
from monai.transforms import (
    AddChannel,
    Compose,
    Transform,
    ScaleIntensity,
    ToTensor,
)

import bentoml
from bentoml.frameworks.pytorch import PytorchModelArtifact
from bentoml.adapters import FileInput, JsonOutput
from bentoml.utils import cached_property

MEDNIST_CLASSES = ["AbdomenCT", "BreastMRI", "CXR", "ChestCT", "Hand", "HeadCT"]

class LoadStreamPIL(Transform):
    """Load an image file from a data stream using PIL."""

    def __init__(self, mode=None):
        self.mode = mode

    def __call__(self, stream):
        img = Image.open(stream)

        if self.mode is not None:
            img = img.convert(mode=self.mode)

        return np.array(img)

@bentoml.env(pip_packages=["torch", "numpy", "monai", "pillow"])
@bentoml.artifacts([PytorchModelArtifact("classifier")])
class MedNISTClassifier(bentoml.BentoService):
    @cached_property
    def transform(self):
        return Compose([LoadStreamPIL("L"), AddChannel(), ScaleIntensity(), ToTensor()])

    @bentoml.api(input=FileInput(), output=JsonOutput(), batch=True)
    def predict(self, file_streams: List[BinaryIO]) -> List[str]:
        img_tensors = list(map(self.transform, file_streams))
        batch = torch.stack(img_tensors).float()

        with torch.no_grad():
            outputs = self.artifacts.classifier(batch)
        _, output_classes = outputs.max(dim=1)

        return [MEDNIST_CLASSES[oc] for oc in output_classes]

# -

# Now the script is loaded and the classifier artifact is packed with the network's state. This is then saved to a repository directory on the local machine:

# +
from mednist_classifier_bentoml import MedNISTClassifier

bento_svc = MedNISTClassifier()
bento_svc.pack('classifier', net.cpu().eval())

saved_path = bento_svc.save()

print(saved_path)
# -

# We can look at the contents of this repository, which includes code and setup scripts:

# !ls -l {saved_path}

# This repository can be run like a stored program where we invoke it by name and the API name ("predict") we want to use and provide the inputs as a file:

# !bentoml run MedNISTClassifier:latest predict --input-file {image_files[0][0]}

# The service can also be run off of a Flask web server. The following script starts the service, waits for it to get going, uses curl to send the test file as a POST request to get a prediction, then kill the server:

# + magic_args="-s {image_files[0][0]}" language="bash"
# # filename passed in as an argument to the cell
# test_file=$1
#
# # start the Flask-based server, sending output to /dev/null for neatness
# bentoml serve --port=8000 MedNISTClassifier:latest &> /dev/null &
#
# # recall the PID of the server and wait for it to start
# lastpid=$!
# sleep 5
#
# # send the test file using curl and capture the returned string
# result=$(curl -s -X POST "http://<IP_ADDRESS>:8000/predict" -F image=@$test_file)
# # kill the server
# kill $lastpid
#
# echo "Prediction: $result"
# -

# The service can be packaged as a Docker container to be started elsewhere as a server:

# !bentoml containerize MedNISTClassifier:latest -t mednist-classifier:latest

# !docker image ls

if directory is None:
    shutil.rmtree(root_dir)

This issue is kind of related: https://github.com/mwouts/jupytext/issues/723. Unfortunately, jupytext doesn't have the option to not convert non-code cells, let alone a way of not converting certain code cells, which would be useful in this case.

Potential solution for Monday: https://github.com/mwouts/jupytext/issues/513#issuecomment-683458030.

OK let's see if that works, if not we can remove the writefile cells and add the files into the repo, I had wanted to avoid this so the notebook would work on Colab.

Yeah that's a good reason for leaving it in the notebook. We don't want our checks to be hampering our code anyway.

I couldn't push to your branch, but merging with my PR seemed to solve the problem. I had to add # noqa: E402 to the line from mednist_classifier_bentoml import MedNISTClassifier, and there was an unrelated problem that needs reticfying.


swift-2.0-minimal-changes branch breaks with Swift 2.2

Compiler complains of redundant CustomDebugStringConvertable declaration on AnyPromise extension in AnyPromise.swift

Thanks for the notice. Though, the minimal changes branch was never meant to be a continually supported version of PK, I'm sorry to say. It was mostly provided as a convenience to prevent forcing users to adopt all changes of PK3 right away. If it is now broken, it's unfortunate, but it's unlikely that one of us will get to it unless there is an outpouring of requests or some greater need emerges. And in fact, any updates would likely need to go on a branch off this one, as updating to support Swift 2.2 would break compatibility for anyone relying on this branch for Swift 2.0 support. Of course if someone were to provide a PR, we could merge it into a new swift-2.2-minimal-changes branch, but officially it's not something that is in scope.
Again, apologies. But due to already-strained developer-hours available from primary contributors, our focus will likely remain on the critical branches for the foreseeable future. I'll update the documentation to indicate that the branch is Swift 2.0/2.1 only.

Someone created a fork here with the (very small) change required to fix this error.
Fork: https://github.com/SLab7/PromiseKit/tree/swift-2.0-minimal-changes
Commit: https://github.com/SLab7/PromiseKit/commit/1ffca8f0bb5dbce68d7f921a208527896b98a1fd

@brianhv thanks for pointing us to this. I've messaged him to ask if he would like to open a PR. If not, I guess I'll just apply this myself after all as it is quite tiny, assuming nothing else needs to be done.


(PUP-7815) Add Task class, have Modules find their Tasks

Add a new class to represent tasks contained in modules. There's not
much functionality in it yet: it can find valid task files in a module's
tasks directory and construct tasks composed of them, where the tasks
can't do anything but tell you their name, their paylod files, their
metadata file (if they have one), and the module they belong to, all via
attribute reader instance methods. There are no other instance methods,
but there are a few public class methods for validating task names and
file names.
When modules are created, they check to see if their tasks directory
exists. If so, they use the Puppet::Module::Task class to create a Task
object for every task that exists in their task directory.

Hmm, I don't understand why the AppVeyor build failed -- there were some warnings, but the specs all passed, and I can't find the exception it claims occurred. Previous AppVeyor builds for this PR passed, and the only changes have been a small amount of new behavior around names that should be the same on all platforms. Is there a way to kick the AppVeyor build? Or can you see the exception?

CLA signed by all contributors.

I think tasks should be represented in modules as Data types. Then there is no need to write any code at all to load them, reflect them, serialize them, a name can directly refer to a task, etc. etc. I think invention of yet more special kinds of things with new ways of loading them is going in the wrong direction.
The description of how the logic in this PR seems to say that eager loading of tasks will take place - that does not sound good at all if that is always the case at runtime. Is that for reflection? That should be solved a different way.
-1 to the design.

Originally when we looked at the module code we thought we could add tasks as just another file type for modules, but since the filetype methods operate directly on files present on disk while there is some logic necessary to map between available tasks and files on disk, since there is not a 1:1 correspondence like with a manifest. In the near future we'll have more, say for loading the task metadata and gathering the pieces of file metadata we want to serve through the API. A separate class that contains only this logic and whose state acts as a caching mechanism seemed like a sensible way to implement this.
If what you're objecting to is the separate class, then I suppose we could move this logic into the the module class. If you have something different in mind (which I suspect from your usage of capital-D 'Data'), could you elaborate on that?
As for the eager loading of tasks: I think this is desirable, and was motivated by performance considerations. This lets us cache the list of available tasks by using the environment cache in Puppet Server, since the environment will load its modules will load their tasks, after which listing them involves no IO, and the list should be correct for as long as the environment is in the cache.
Although, as I say this, I realize that assumes code manager controls environments; are there no guarantees at all about the consistency of environment contents without that? Should the implementation always read the currently available tasks from disk whenever modules are asked for their tasks, and the caching implemented some other way? Or is it acceptable for the loading occur the first time that a module is asked for its tasks? (The reason I felt is was alright to eagerly load is that tasks already eagerly load their metadata from disk when they're initialized, so I thought that a directory check for each module would not significantly increase the load time.)

Based on your discussion with Michael, it seems like you're concerned with how Tasks will be represented in Puppet code. The changes in this PR are intended only to provide data about standalone task files in modules that we'll serve through the API; it shouldn't define anything about the language side of Tasks or how users will define or load them. If you think it does, please explain how and provide a sketch of an alternative implementation that allows us to serve these standalone tasks without requiring them to be defined in Puppet code.
As the language changes to allow defining and using tasks in Puppet code are made, I think we should be able to integrate those tasks into the module's notion of what tasks are available for serving through the API. Do you disagree?

I think tasks should be represented in modules as Data types.

In the future I'd like to move task file serving out of puppet ruby and just serve these files via puppet server. I understand there's some knowledge about module path we'd need to create but thats ok. So I'm hesitant to tie task implementation into puppet very closely.

I've updated this to avoid loading any tasks before the tasks method of a Module instance is called.

Closing in favor of #6120


When the scroll is decelerating in debug mode, I cannot press the navigation bar buttons.

When the scroll is decelerating in debug mode, I cannot press the navigation bar buttons (right or left buttons).
Actually, the button changes to pressed state, and when the scroll stops, the action of the button is launched. It's very strange.
However, in release mode, it works perfectly.
BTW this is part of my configuration:
_containerController.maximumLeftDrawerWidth = 270.0f;
_containerController.bezelPanningCenterViewRange = 85.0f;
_containerController.openDrawerGestureModeMask = MMOpenDrawerGestureModeAll;
_containerController.closeDrawerGestureModeMask = MMCloseDrawerGestureModeAll;
_containerController.centerHiddenInteractionMode = MMDrawerOpenCenterInteractionModeNone;
Any idea?

The problem is due to FLEX debugging library. Sorry.


Exception: Error while deserializing header: HeaderTooLarge

Header to large error when running example.  I suspect this is a problem with how I downloaded the models when running prep scripts.  Will retry from scratch.
11:32 $ python inference_magictime.py --config sample_configs/RealisticVision.yaml
The results will be save to outputs/RealisticVision-3
Use MagicAdapter-S
Use MagicAdapter-T
Use Magic_Text_Encoder
Traceback (most recent call last):
  File "myhome/dev/repos/MagicTime/inference_magictime.py", line 249, in <module>
    main(args)
  File "myhome/tools/miniforge3/envs/magictime/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/dkords/dev/repos/MagicTime/inference_magictime.py", line 61, in main
    text_encoder = CLIPTextModel.from_pretrained(model_config.pretrained_model_path, subfolder="text_encoder").cuda()
  File "myhome//tools/miniforge3/envs/magictime/lib/python3.10/site-packages/transformers/modeling_utils.py", line 3284, in from_pretrained
    with safe_open(resolved_archive_file, framework="pt") as f:
safetensors_rust.SafetensorError: Error while deserializing header: HeaderTooLarge

Verified: https://huggingface.co/runwayml/stable-diffusion-v1-5/tree/main/text_encoder
seems to not download properly on my linux ubuntu 22.04.  File sizes are wrong.  Short investigation indicates HeaderTooLarge error is likely a problem with git-lfs missing or the actual large blobs not downloading.
This is due to a system misconfiguration in my case.  git lfs install command by itself doesn't seem to work.  Further investigation yields some questions about the correct way to actually install git lfs.  Will attempt to just install via package manager.

Resolved with proper git-lfs installation before running clone.


initialization broken under Aspace v2.4.0 ?

ArchivesSpace v2.4.0 fails to start for me with  this plugin installed. ( Both with current master and with 2.1.4, which is what was initially installed. )
Initialization complains about duplicate versions of pry specified, but I can only find one version installed or referenced in Gemfiles. No pry in archivesspace/gems/gems/...  .
I tried removing the '~> 0.11.3' from Gemfile and re-running initialize-plugin script, but then it complains about not finding slop gem.
( Duplicate version message said the versions were 'pry',  '~> 0.11.3' and 'pry'  )

INFO: An exception happened during JRuby-Rack startup

[!] There was an error parsing `Gemfile`: You cannot specify the same gem twice with different version requirements.
You specified: pry (>= 0) and pry (~> 0.11.3). Bundler cannot continue.

 #  from /projects/Archivespace/archivesspace/data/tmp/jetty-<IP_ADDRESS>-8089-backend.war-_-any-/webapp/WEB-INF/Gemfile:58
 #  -------------------------------------------
 #  require 'asutils'
 >  ASUtils.load_plugin_gems(self)
 #  -------------------------------------------
--- System
jruby <IP_ADDRESS> (2.3.1) 2017-03-06 90fc7ab Java HotSpot(TM) 64-Bit Server VM 25.172-b11 on 1.8.0_172-b11 +jit [darwin-x86_64]
Time: 2018-06-08 12:50:36 -0400
Server: jetty/8.1.5.v20120716
jruby.home: uri:classloader://META-INF/jruby.home

--- Context Init Parameters:
jruby.compat.version = 1.9
jruby.max.runtimes = 1
jruby.min.runtimes = 1
public.root = /
rack.env = production

--- Backtrace
Bundler::Dsl::DSLError:
[!] There was an error parsing `Gemfile`: You cannot specify the same gem twice with different version requirements.
You specified: pry (>= 0) and pry (~> 0.11.3). Bundler cannot continue.

 #  from /projects/Archivespace/archivesspace/data/tmp/jetty-<IP_ADDRESS>-8089-backend.war-_-any-/webapp/WEB-INF/Gemfile:58
 #  -------------------------------------------
 #  require 'asutils'
 >  ASUtils.load_plugin_gems(self)
 #  -------------------------------------------

     eval_gemfile at /projects/Archivespace/archivesspace/gems/gems/bundler-1.16.1/lib/bundler/dsl.rb:53
         evaluate at /projects/Archivespace/archivesspace/gems/gems/bundler-1.16.1/lib/bundler/dsl.rb:12
            build at /projects/Archivespace/archivesspace/gems/gems/bundler-1.16.1/lib/bundler/definition.rb:35
       definition at /projects/Archivespace/archivesspace/gems/gems/bundler-1.16.1/lib/bundler.rb:135
            setup at /projects/Archivespace/archivesspace/gems/gems/bundler-1.16.1/lib/bundler.rb:101
           <main> at /projects/Archivespace/archivesspace/gems/gems/bundler-1.16.1/lib/bundler/setup.rb:20
          require at org/jruby/RubyKernel.java:961
          require at uri:classloader:/META-INF/jruby.home/lib/ruby/stdlib/rubygems/core_ext/kernel_require.rb:133
           <main> at /projects/Archivespace/archivesspace/data/tmp/jetty-<IP_ADDRESS>-8089-backend.war-_-any-/webapp/WEB-INF/app/main.rb:1
          require at org/jruby/RubyKernel.java:961
          require at uri:classloader:/META-INF/jruby.home/lib/ruby/stdlib/rubygems/core_ext/kernel_require.rb:55
  block in (root) at /projects/Archivespace/archivesspace/data/tmp/jetty-<IP_ADDRESS>-8089-backend.war-_-any-/webapp/WEB-INF/app/main.rb:1
    instance_eval at org/jruby/RubyBasicObject.java:1687
           (root) at /projects/Archivespace/archivesspace/data/tmp/jetty-<IP_ADDRESS>-8089-backend.war-_-any-/webapp/WEB-INF/config.ru:1
           <main> at /projects/Archivespace/archivesspace/data/tmp/jetty-<IP_ADDRESS>-8089-backend.war-_-any-/webapp/WEB-INF/config.ru:4
     start_server at launcher/launcher.rb:92
             main at launcher/launcher.rb:157
           <main> at launcher/launcher.rb:261

--- RubyGems
Gem.dir: /projects/Archivespace/archivesspace/gems
Gem.path:
/projects/Archivespace/archivesspace/gems
/projects/Archivespace/archivesspace/plugins/aspace-import-excel/gems
Activated gems:
  did_you_mean-1.0.1
  bundler-1.16.1

--- Bundler
Bundler.bundle_path: /projects/Archivespace/archivesspace/gems
Bundler.root: /projects/Archivespace/archivesspace/data/tmp/jetty-<IP_ADDRESS>-8089-backend.war-_-any-/webapp/WEB-INF
Gemfile: /projects/Archivespace/archivesspace/data/tmp/jetty-<IP_ADDRESS>-8089-backend.war-_-any-/webapp/WEB-INF/Gemfile
Settings:
  build.charlock_holmes = --with-icu-dir=/usr/local/opt/icu4c --with-cxxflags='-Wno-reserved-user-defined-literal -std=c++11'
  build.puma = --with-opt-dir=/usr/local/opt/openssl
  without = [:test, :development, :doc]

--- JRuby-Rack Config
compat_version = RUBY1_9
default_logger = org.jruby.rack.logging.StandardOutLogger@59b447a4
equals = <error: pid 9603 exit 0>
err = java.io.PrintStream@a826ff8
filter_adds_html = true
filter_verifies_resource = false
ignore_environment = false
initial_memory_buffer_size =
initial_runtimes = 1
jms_connection_factory =
jms_jndi_properties =
logger = org.jruby.rack.logging.ServletContextLogger@49889154
logger_class_name = servlet_context
logger_name = jruby.rack
maximum_memory_buffer_size =
maximum_runtimes = 1
num_initializer_threads =
out = java.io.PrintStream@45d7495e
rackup =
rackup_path =
rewindable = true
runtime_arguments =
runtime_environment =
runtime_timeout_seconds =
serial_initialization = false
servlet_context = ServletContext@o.e.j.w.WebAppContext{/,file:/projects/Archivespace/archivesspace/data/tmp/jetty-<IP_ADDRESS>-8089-backend.war-_-any-/webapp/},/projects/Archivespace/archivesspace/wars/backend.war
throw_init_exception = false

Jun 08, 2018 12:50:36 PM org.eclipse.jetty.server.handler.ContextHandler$Context log
WARNING: ERROR: initialization failed
org.jruby.rack.RackInitializationException:
[!] There was an error parsing `Gemfile`: You cannot specify the same gem twice with different version requirements.
You specified: pry (>= 0) and pry (~> 0.11.3). Bundler cannot continue.

sigh< OK, I finally removed hints of "pry" from the code and the Gemfile.

Please try V2.1.8 (https://github.com/harvard-library/aspace-import-excel/releases/tag/v2.1.8 )
Note that we don't yet have 2.4 installed, so no guarantees, but hopefully this should work!
Let me know,
Cheers,
Bobbi

plugin v2.1.8 starts up successfully under aspace v2.4.0 .
( I haven't run it long enough to verify there are no other issues after startup. )
It appears to me that there is an issue with  group :development, :test do  and  bundler.
If I comment out those blocks in  backend|frontend|public Gemfile's ,  and build distribution, previous plugin versions load with that build.
If I comment out the group :development, :test do and following end lines to force the gems to be included in distribution build, previous plugin versions also load.
But for some reason, those conditional blocks seem to cause some confusion for the bundler.
It thinks multiple versions are installed, but it can't seem to find them.
Those conditional blocks were recently changed between space v2.3.2 and v2.4.0

sigh< OK, I finally removed hints of "pry" from the code and the Gemfile.

Please try V2.1.8 (https://github.com/harvard-library/aspace-import-excel/releases/tag/v2.1.8 )
Note that we don't yet have 2.4 installed, so no guarantees, but hopefully this should work!
Let me know,
Cheers,
Bobbi

Gemfile issues in the core code need to be referred to the Core Committers, or through the JIRA board.
I'm closing this issue as "fixed" for this plugin.
Thanks,


RDKitValidation doesn't catch some valence issues

Describe the bug
The RDKitValidation class doesn't return an error for some valence violations that are not detected by Atom::calcExplicitValence.
To Reproduce
In [1]: from rdkit import Chem

In [2]: from rdkit.Chem.MolStandardize import rdMolStandardize

In [3]: vm = rdMolStandardize.RDKitValidation()

In [4]: mol = Chem.MolFromMolBlock('''
   ...:   MJ240300
   ...:
   ...:   2  1  0  0  0  0  0  0  0  0999 V2000
   ...:    -4.8993    1.8410    0.0000 Br  0  5  0  0  0  0  0  0  0  0  0  0
   ...:    -5.6138    1.4285    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   ...:   2  1  1  0  0  0  0
   ...: M  END
   ...: ''', sanitize=False)

In [5]: vm.validate(mol)
Out[5]: []

Expected behavior
The validation should fail with an error message like Explicit valence for atom # 0 Br greater than permitted
Configuration (please complete the following information):

RDKit version: current master branch
OS: [e.g. Ubuntu 20.04] Fedora 40
Python version (if relevant): not relevant
Are you using conda? no
If you are not using conda: how did you install the RDKit? from source code

Additional context
The expected behavior would be easily produced if, in addition to atom->calcExplicitValence(), RDKitValidation also called atom->calcImplicitValence() (just like I think Atom::hasValenceViolation does). If this is the desired solution, I would be happy to create a small PR implementing it, but I found it a bit surprising that an "Explicit valence .." error was reported by Atom::calcImplicitValence and is not detected by Atom::calcExplicitValence and wondered if I am missing something.
Moreover, the behavior apparently depends on the input format, and C[Br-] is sanitized without errors.

Just a reference point: the current release version (2024.03.6) does generate the expected warnings here.


Show master page programmatically?

Is there a method to switch to the master view in a master-detail layout via code? I am not displaying the nav bar but want to trigger this. Effectively the equivalent to Xamarin's native:
Ispresented = true
Also, is there a way to replace the root with a new master-detail viewmodel, as in the case where you navigate from a Login screen to a Landing screen with a Menu? For example:
CoreMethods.PushMasterDetailModel<???>();

You dont have use default themedmastedetaiilnavigationcontainer class from freshmvvm. you can create a new one and do whatever you want with it. it is totally independent implementation than freshmvvm itself. see how I have done here.
https://github.com/rid00z/FreshMvvm/issues/188


Add option to choose which side of text in the button the spinner shows up in

Is it possible to add an option to configure the spinner location? either float left or float right?
supporting the bootstrap style ``// Classes .pull-left {   float: left !important; } .pull-right {   float: right !important; }
If the outer div style for in the Spinner.jsx added the float: right !important or float: left !important with some padding, it would allow us to choose which side of the text the spinner shows up.

@feiwaltan this is not supported as for now but can be easily extended. You may file a PR and I will either merge or help to make it production-ready.

@yury-dymov can you provide some instructions on how to contribute to the project? I'd like to add this ability and need some guidance on building and testing out the update.

@feiwaltan should pretty straightforward, you need patch https://github.com/yury-dymov/react-bootstrap-button-loader/blob/master/src/ButtonLoader.jsx by adding a prop, which would apply a desired style to the Spinner element if equals true

@yury-dymov do I need branch creation permission to be able to contribute? I've got changes ready to commit to the two files that would be modified.

fork the project
start your branch in your repo
add some commits
create a pull request from your branch to my repo on the github
I will review and ask for more changes or merge right away


Update Composer dependencies (2022-04-03-00-02)

Visual regression test passed!
View the visual regression test report


Unable to log in with VSCODE_CLIENT_ID VSCODE_TENANT scope

Testing #226539
Using the latest VS Code Insiders on Ubuntu with the Fugio extension and my personal Microsoft account.
I tried putting in the scope VSCODE_CLIENT_ID:a4fd7674-4ebd-4dbc-831c-338314dd459e VSCODE_TENANT:common profile and got the error "invalid_request: The provided value for the input parameter 'redirect_uri' is not valid. The expected value is a URI which matches a redirect URI registered for this client application."
I then tried putting in the scope VSCODE_CLIENT_ID:a4fd7674-4ebd-4dbc-831c-338314dd459e, VSCODE_TENANT:common, profile and got the error "Specified tenant identifier 'common,' is neither a valid DNS name, nor a valid external domain." which seems to imply that the values should not be comma-separated, either.

Got the same error

This should be working now. I forgot to add a redirect uri to the client id configuration

Steps:

Install Fugio
Run Fugio: Mint Token
microsoft
VSCODE_CLIENT_ID:a4fd7674-4ebd-4dbc-831c-338314dd459e VSCODE_TENANT:common profile

It should take you through the flow to mint a token


using tf.function on a model call will not utilize gpu nor VRAM but log placement to gpu

Click to expand!
Issue Type
Bug
Source
source
Tensorflow Version
2.9.1
Custom Code
Yes
OS Platform and Distribution
Windows 11
Mobile device
No response
Python version
3.9.12
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
11.2/8.6
GPU model and memory
3090 24GB
Current Behaviour?
Utilize tf.function on a function that calls model(a,b) (aka model.call)
Although eager tensors have all been allocated to the gpu, actual training doesnt happen on the gpu, rather the cpu. Notice 20x slower training speeds than when utilizing nested tf.functions in model.call, which is not allowed in MirroredStrategy.

Standalone code to reproduce the issue
Model.call:

def call(self,x,y):
    vals=self.disc_gradients(x,y)
    self.gen_gradients(x,y)
    return vals

model call wrapper:
@tf.function#this is the problematic tf.function, if this tf.function is moved to the below tf.functions it works fine.
def run_batch(self,batch_x,batch_y):
    val=self.model(batch_x,batch_y)
    return val

gradients functions:
<EMAIL_ADDRESS>def get_disc_gradients(self,x,y):
    #calc val
    self.disc_optimizer.apply_gradients(disc_gradients,vars)
    return disc_gradients

Obviously this code currently does nothing, it's a minimal reproducible example.

### Relevant log output

```shell
WARNING:tensorflow:Using MirroredStrategy eagerly has significant overhead currently. We will be working on improving this in the future, but for now please wrap `call_for_each_replica` or `experimental_run` or `run` inside a tf.function to get the best performance.

The above log is relevant as it tries to make us wrap the entire model.call in a tf.function, which is where I discovered this entire issue. This issue is replicateable w/o using MirroredStrategy: all that matters is that tf.function is wrapping a model call.
No other relevant log output could be found.
Only way to notice the issues are the extremely long batch times, along with
the GPU only utilizing 3GB of VRAM compared to a full 20gb when the tf.function is moved.

To be more specific, wrapping a training loop for a nested keras.model in any way will cause this issue. Wrapping each individual gradients functions in @tf.function prevents this issue, but is bad since we can't use mirroredstrategy.

It appears @tf.function placement on nested models matters greatly for gradient calculation: the lack of GPU vram usage is from missing gradients- although it doesn't explain the 700s batch times and clear cpu usage over GPU. Something is seriously wrong here, haha.

@Apprisco,
I tried to execute the provided code. Kindly find the gist of it here.  The code provided is not complete hence it would be difficult for us to pinpoint the issue.
Could you share complete stand alone code to replicate the issue or a colab gist with the error reported. Thank you!

It wouldn't be ideal to share my current code as it is for an upcoming research paper, it may take a bit but let me produce a minimal reproducible example.

Update: turns out my nested models were not models but layers: this is still an issue regarding tf.function, but different scope.

Issue went away when we changed to actually using nested models, not gigantic custom layers. I still believe this be an issue but this is no longer one that concerns us. Especially since we moved on to horovod, not mirroredstrategy.

Error has returned, but again when using nested layers not models haha


Cookies and session not available for flutter dekstop windows

Je n'arrive pas à faire préciser les cookies et session dans la webview

About session,
the official package webview_flutter also not provide session-related methods,
and I will follow this API and wont provide session-related functions.
About cookie,
since there are some news about web phishing by webview2 cookies,
of course I think most of us won't do that (phishing),
however I won't provide any cooke get / set methods in this package.
So this issue won't fix


ShirukaServer.java:201: Load plugins here.

The puzzle 1-7308ca1b from #1 has to be resolved:
https://github.com/shiruka/shiruka/blob/ff7539ba30326ff3a7d14c6f861cbab6de82e224/src/main/java/io/github/shiruka/shiruka/ShirukaServer.java#L201-L201
The puzzle was created by @portlek on 26-Dec-20.
Estimate: 60 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 1-7308ca1b has disappeared from the source code, that's why I closed this issue.


Terminal as a tab

Maybe it is an idea to show the terminal as a tab in atom.

@JostCrow I appreciate the idea. I'll put it on a list of possible features to add to the package.

Ok. Maybe I can see if I could work something out for this one.

:+1: would really appreciate this, takes a lot of horizontal space otherwise.

+1

Sorry for the delay. v0.12.0 now has support for tab views! Just make a terminal as usual then drag your status icon to the tab-bar and it will become a tab. If the status icon has a name (via status-icon rename) then it will adopt that name for the tab.

Thanks. I tested it as soon as I saw it. It looks great and works fine. Thanks.

Thank you, really grateful for this feature.

Glad you guys like it. It is experimental for now since I'm finding bugs here and there. Please use it extensively and report back if you find any issues.

I still have 1 request for this package. It would be awesome it there would be a command to add a terminal tab in the active pane.

That can be done. I'll add a command for that and let you know when it has been released.

I found an issue with this: https://github.com/jeremyramin/terminal-plus/issues/77

Issue #91 is related to this.

This feature is fantastic enough to be on the features list IMO ;P

+1

@jeremyramin any updates or eta on the feature? thanks :)

Super excited to have tabbed terminal, this package is phenominal and the only thing i wanted was to save vertical space on my laptop screen, you sir have made this dream possible. A+

Thank you, super useful.

Very useful! But I had to stumble upon this issue to find out that it was supported.

I still have 1 request for this package. It would be awesome it there would be a command to add a terminal tab in the active pane.

That can be done. I'll add a command for that and let you know when it has been released.

That would be very useful.


26 doc update documentation for UI setup and development

Summary
Documentation enhancement covering contributing, API client, and UI design.
Any fixes or recommendations are welcome. ❤
Types of changes

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

Testing
N/A
Notes
N/A

I guess I should start marking PRs as drafts to avoid the wall of shame like this 😅

I guess I should start marking PRs as drafts to avoid the wall of shame like this 😅

There's no shame in healthy collaboration, @PlesnikJakub. ❤ #oneteam


One more fallback font for Chinese

Added SimSun as extra fallback font for Chinese and default language.

CI fails for some reason, I will check why.


Loading game crashes

Describe the Bug
Loading game crashes minecraft1.18.2
Reproduction Steps
1.minecraft 1.18.2
2.Loading game crashes
3.
...
Expected Result
minecraft-exported-logs-2022-08-15T10-57-47.log
Screenshots and Videos
No response
Crash Report or Log
No response
Operating System
10
What is your GPU?
rtx3060
Mod Version
0.6.5
Minecraft Version
1.18.1
Loader Version
forge40.1.73
Other Mods
did not see 1.18.2 So choose 1.18.1
Additional Context
No response

The mod is supposed to work for 1.18.2 just fine and has been then 0.6.5 came out and it crashes upon launch, so going to stick to 0.6.4 until this is resolved.

该 mod 应该适用于 1.18.2 就好了，然后 0.6.5 出来了，它在启动时崩溃了，所以要坚持使用 0.6.4 直到这个问题得到解决。

ok

Same as #148