Using existing rendered template

What problem does this feature solve?
Can I mount vue on existing php rendered markup so vue can directly hook on it.
example : If direct request then server render the markup ( without using v8 ) and vue will take over once document loaded at client side  and if  vue router request then js will render the markup. I can also try that both php and js will render same markup ?
What does the proposed API look like?
If php rendered markup with data-rendered attribute is detected then vue will hook on it directly.

Vue server side rendering requires v8, but you can call v8 from php via v8js: https://gist.github.com/yyx990803/9bdff05e5468a60ced06c29c39114c6b#environment-agnostic-ssr

@yyx990803 I may be completely wrong but please try to understand. I tried calling v8 from php. But I am talking about existing template. Projects that are already doing well with php. They are already server rendered, In such case can I hook vue component directly on it ? I can sync markup in js ( vue component ) so it can work well with vue router. In short server side rendering without v8 dependancy. Consider this scenario where project is going to distribute among several client and we can't force everyone to install php v8 js extension.

I'm not sure if I understand what you mean. You obviously can mount Vue components to existing markup, it simply replaces whatever is previously there.

Yes mount to existing markup and instead of replace can we do sync ? Like markup + state to sync with vue component.

@yyx990803 vue-server-renderer allows the generation, server side, of an HTML template that will later be used by the client side code (so it can properly hydrate).
It does so by the addition of  data-server-rendered="true" in the main div.
Is there a documentation which explains how vue-server-renderer renderToString works? Which kind of markup it creates?
With this, maybe it should be possible to generate manually the same markup in non-nodejs env, without the use of v8js (or something else), just with a basic template language.
This may seems odd, but some code may want to use ssr without node and without any tools – just with plain old markup.
If it's not too complicated.
(I may understand it will depends on vuejs version, but still).
Thanks a lot.


Unable to register through UI

Hi, on a recent attempt to register a subdomain, it failed to login. I am unsure of its default behavior as this is the first time I am checking it out. After approving the sign-in it continued to remain in the same page. It did show it was redirecting to auth.is-a.dev. Attached is an error screenshot.

@andrewstech

Due to the amount of bugs I have taken the decision to take down the site at this time. I'm currently working on a Full rewrite that I hope to release by the end of the week


Update ReproduceInfoPrinter.java

to remove the additional double quotes in jvm argline in reproduce with

Fix for issue:
https://github.com/elastic/elasticsearch/issues/15048


added tests for a real 5th Monday of a month and for a nonexistant 5th Monday

I proposed this at the end of last year based on what I was seeing in others' solutions (see discussion here https://github.com/exercism/exercism.io/issues/2142) and @kytrinyx gave the go-ahead.
Let me know what you think. Thanks!

Nice idea, thanks!


Chore: Python 2 deprecation in lib

Changelog Description
Remove Python 2 compatibility from functions in ayon_core.lib.
Additional info
Classes do no inherit from object. Simplified doubled imports for Py2 and Py3 compatibility of configparser and lru_cache. Use WeakMethod directly from weakref and warn on import of python_2_comp.py.
Testing notes:

Go through code and check possible errors I made.
Validate everything is working as expected.

Remove python_2_comp.py file now

We have to be somewhat careful about breaking backwards compatibility. I can't know if there is an addon that is using it, having it there with warning for some time won't hurt.

their logic wouldn't be valid for Py3 anyway

It would be Py3 valid. The content was Py2 and Py3 compatible, you could import it to support both.
Note, start filename with underscore if should not be imported publicly.


Revert rust color change

The previous change #4319  made it hard to distinguish the language and made it hard to recognize for all users that are used to the old color. Practically speaking the change introduced more problems than the esthetics it wanted to solve gained us. Also esthetics are personal opinions, so I'd leave a new color up to the rust team to decide.

I am in favour of this. As has been mentioned in the comments of #4319 the deeper red colour is harder to distinguish from other languages (D, Ruby, Scala, etc.) while the original colour worked just fine. Also I do not believe that the #a62c00 shade has any real association with the Rust community.

FWIW, I started a humorous thread over here discussing the change. It seems like most people favor the old color, although I don't have as strong of an opinion on this subject. I slightly favor the old color.

I am in favor of this as well. The color that people would most associate with Rust would be that of its unofficial mascot, which is orange: http://rustacean.net
Additionally, the rationale that the old color doesn't look like iron oxide is misguided since the language was named after the rust fungus, which has an orange color: https://en.wikipedia.org/wiki/Rust_(programming_language)#History

I, too, prefer the older colour. Please bring it back. Annoying that this change was done without the O.K. of the Rust team.

I think Rust color is a better choice.

I, too, prefer the older colour. Please bring it back!!!!!!!!!!!

I think the new color is 10000x better than the old one. The color suggested by @quininer could be a nice compromise though.

OLD COLOR BACK !!!!!!!!!!!

Otherwise, I will delete all my repos and move to gitlab.

the new colour is great and definitely better than the old one!

I am in favor of the old one as well. It's too dark and distinguishable from other languages.

I'll try get this out today.

This is now live on GitHub.com.


feat: miner response times

Description
This PR fixes #122.  It sets up the validator forwarder to use said processing times in it's rewarding/scoring of miners.  This will be used alongside data validation to provide a weighted score. masa/validator/forwarder.py now includes the process time for each valid response (described as latency currently).  note that it's impact on a score has not been implemented yet.  waiting for https://github.com/masa-finance/issues/issues/197 to be implemented, and then we can use the response time as a weighted param towards the score (perhaps data validation is 0.75 weight and response time is 0.25 weight).
Notes for Reviewers

will wait for the work done by https://github.com/masa-finance/issues/issues/197 to complete the implementation

Signed commits

[x] Yes, I signed my commits

@hide-on-bush-x I was thinking we can now take your list of valid responses, and further score them based on these response times.  I don't think it makes sense to consider how fast a non-valid response was, otherwise miners can send back bad data fast, and get at least some rewards, which we don't want, right?

Yeah that makes sense, Ill take a look at how to achieve that

Closing this pull request as this functionality has already been merged into main via the data validation PR


Reverted commit 90f2daf

Hi Mark, is there a reason 90f2daf was reverted? Thanks.

Revert here: https://github.com/markbates/refresh/commit/25435da78dbf2fa5067f3c156051103ad72bce0e

Yes; it was causing significant CPU usage. Reverting this commit brought CPU usage from 100+% to 0.3% on my machine. A lot of users complained about the increased CPU usage.

You've reverted two commits, the one I'm referring to above seems unrelated to CPU usage. The CPU usage commit you reverted is here: https://github.com/markbates/refresh/commit/0b2bb8035576fb5fffcee9eed3233be1abdc9098

That commit needed to be reverted twice due do a bad merge.

Mark Bates
On Jul 12, 2017, 9:58 PM -0600, nullbio<EMAIL_ADDRESS>wrote:

You've reverted two commits, the one I'm referring to above seems unrelated to CPU usage. The CPU usage commit you reverted is here: 0b2bb80
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

This is somewhat confusing, because you've removed two different features and one is unrelated to CPU usage. Even if it needed to be reverted twice you definitely reverted two separate pieces of code.
Just to clarify again, this is the commit I'm referring to which was merged a while ago: https://github.com/markbates/refresh/commit/90f2daf12da08f62255f309ece774fa94604a20f which has now seemingly mistakenly been reverted since it has nothing to do with CPU usage.
The one that seems intentionally reverted is here: https://github.com/markbates/refresh/commit/0b2bb8035576fb5fffcee9eed3233be1abdc9098 which has a commit message that refers to huge CPU usage.

Hey @markbates not sure if you saw my reply. Just trying to get this sorted as quick as possible.

I have. I'm at GopherCon and have been very busy. Sorry. I will try and sort this out tomorrow.

@nullbio ok, it's back in there. sorry about that.

@markbates Thank you Mark.


.Net 4.0 build?

I see that the .Net 4.0 build was disabled last year, is it possible to bring that back?

Hi @petersondrew ,
I will take look and bring it to back soon.


standard stops to lint inside a Multi-root workspace

What version of this package are you using?
2.0.0
What operating system, Node.js, and npm version?
macOS High Sierra (10.13.6), Node.js 16.8.0, npm 7.21.0
What happened?
I started a new window of vscode, added a project folder with standard installed and just one file (index.js), i can see the linting is working as expected.

However, if I add a new folder (even an empty one) to the vscode window, with File -> Add Folder to Workspace the lint appears to continue to work until I restart the vscode, after that the lint no longer activates again.

What did you expect to happen?
I expected the lint to the index.js to continue to work, by looking at the options I am not sure if I am missing to enable something else, I am just using the default options, perhaps the standard.workingDirectories is needed here? I am not sure what the option is for, tried to use it but it seems it is not relevant to this problem.
Are you willing to submit a pull request to fix this bug?
No, though if someone guides me to where to look in the code, I will happily check if I can fix it

Hey, @bjrmatos! :wave:
Thanks for your report! :smile:
That's the expected behavior, read v2.0.0 changelog for details, as you might use standard in one folder, in another ts-standard and in a third open folder in the workspace eslint.
It might be complicated to auto-detect automatically what engines to use and load them at the same time.
The way to do this currently is to open your 2 folders as you showed on your screen, as it is one folder.
Inside VSCode: File -> Open Folder, and then select your folder containing the 2 subfolders you wanna lint.

I am just using the default options, perhaps the standard.workingDirectories is needed here?

Yes, exactly it is needed in this situation.
Add a .vscode/settings.json file in the root folder you opened with:
{
  "standard.enable": true,
  "standard.enableGlobally": true, /* you would need to install `standard` globally in this case, if you didn't installed it in the root folder, `npm install --global standard` for example. */
  "standard.engine": "standard", /* this could be `standard`, or `standardx` or `semistandard` or `ts-standard` */
  "standard.workingDirectories": ["first-folder", "second-folder"]
}

so the workaround here to have the auto-detect working will be to open just one folder in vscode, and it should be folder that contains the 2 projects, right?

just to know if I understand this clearly, these steps are needed if I decide to open one root folder that contains my 2 projects, right? I think I understand now that in this case standard.workingDirectories is needed to tell vscode-standard to run as is first-folder and second-folder are the root in each case. is this correct?

Yes, it is right! @bjrmatos
Please close this issue, if your problem is solved. :smile:


Inherited and base members doesn't show up if class has virtual functions and base to derived class autocast not working. v0.9.9

Version: 0.9.9 ALPHA
Last fix: d3931fd

Fixed in 7c0ce2ab54ef85fb7a35ae346b35f880eb6861a2. Along with the other fixes will push a new build with this fix included.


Build Error

Helo, I just installed version 1.0.4 because I am using Swift 3.0
I tried to build the project but it gives me error in the Popover.swift file at -  self.showAsDialog(contentView, inView: UIApplication.shared.keyWindow!)
error - 'shared' is unavailable. Use view controller based solutions where appropriate instead.
Please help.

ummm
Can I call UIApplication.shared on your project.
If it can not be called, there is a possibility that it is not swift3.0

I checked and I cannot call UIApplication.shared. Actually I am using this in a custom-keyboard extension and not in the main app. Is that why I am facing this issue?

How can I change it to use in the keyboard extension. any help would be greatly appreciated.

https://developer.apple.com/library/content/documentation/General/Conceptual/ExtensibilityPG/ExtensionOverview.html#//apple_ref/doc/uid/TP40014214-CH2-SW2

Access a sharedApplication object, and so cannot use any of the methods on that object

You may be able to build it if you delete the code below
  open func showAsDialog(_ contentView: UIView) {
    guard let rootView = UIApplication.shared.windows.last ?? UIApplication.shared.keyWindow else {
        return
    }
    self.showAsDialog(contentView, inView: rootView)
  }

  open func show(_ contentView: UIView, fromView: UIView) {
    guard let rootView = UIApplication.shared.windows.last ?? UIApplication.shared.keyWindow else {
        return
    }
    self.show(contentView, fromView: fromView, inView: rootView)
  }

But will it work as expected?

Internally it gets the key window and just calls the following
func show(_ contentView: UIView, fromView: UIView, inView: UIView)

So i can call this function instead of show(_ contentView: UIView, fromView: UIView) to show the popup?

It works 👍 Thank you so much :)

very good!  It may correspond in the future for app extension.


support for orgmode

Org mode (via org babel) allows to include python source blocks. The python code can be edited right away in org mode, but it's a bit of pain to call blacken every time on each code block. (It must be doable in elisp, but for the moment I don't know how to do that). It's much more handy to run blacken-docs on the org file itself.

I don't quite know what to do with the failed checks. In my second to last commit, I repaired the message of pre-commit, namely the long string. For the rest all works on my machine.

That is a messy example, which I also missed.
The problem is that in org mode the noweb references in python code are
written like this: <> and I don't quite know what syntax rules
such code-ids have to follow. Simple rules like checking on letters will
not suffice, because replacing the 5 in your example with a variable name
is still valid code.
I obviously don't want black to change the noweb references, nor the
indentation. Now code ids will not appear (I think) at the right of an
assignment (or something like that), but your example shows that a
multiline criterion is necessary to distinguish between << 5 >> and
<>.
For the moment I don't have a generic solution to tackle your example.
BTW, I apple the hack to my org files, and it works really well. But I
don't write << 5 >> in my code :-)
On Fri, 19 Aug 2022 at 19:21, Anthony Sottile @.***>
wrote:

@.**** commented on this pull request.
In blacken_docs.py
https://github.com/asottile/blacken-docs/pull/168#discussion_r950416002:

@@ -178,6 +184,15 @@ def _latex_match(match: Match[str]) -> str:
code = textwrap.indent(code, match['indent'])
return f'{match["before"]}{code}{match["after"]}'

def _orgmode_match(match: Match[str]) -> str:

   code = textwrap.dedent(match['code'])

   code = re.sub(r'( *<<.*>>)', r'#\1', code)

this hack still isn't going to work -- this is valid code:
x = (
y
<< 5 >> 1
)
—
Reply to this email directly, view it on GitHub
https://github.com/asottile/blacken-docs/pull/168#pullrequestreview-1079136395,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM266OJBZ45DTX2IXRU26LVZ67BBANCNFSM5YH3ZNQQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Feature/new theme

I've tried updating with a new fancier open source theme, which should be more responsive, and more extendable with new features!
Let's hope this upgrade goes smoothly 🙏

yolo


Images are not shown on Readthedocs Website

The first 3 images are not shown on the website:
https://manimce.readthedocs.io/en/latest/examples/hello_example.html
The videos are shown, however.
Any ideas?

I've identified the problem and am working on a fix. Basically, there is a mismatch with the paths.


Yodeling Snowcat - Write ping frames

Allow snowcat to fire an astroid to the planet.
Allow client to write a ping frame to the server.
Here's a video for reference
http://youtu.be/Eu5ZJELRiJ8?t=42s

haha nicely done :smile:. Quick question, what is the use case of sending pings from the client? From my understanding normally the WebSocket server sends the pings and the client send a pong response. Also could you add a section in the README documenting the new method like the other public methods?

So in our use case the server needs to know who is in the background and who is in the foreground to filter our socket messages. We are using a voip background websocket and we don't want unnecessary data sent to the clients while in the background.
Also just for reference, in the RFC 6455 Section 5.5.2 it just specify "endpoint" rather than client or server. We've seen in other client libraries such as socket.io that clients ping and servers pong
@shanksauce is our backend guy and helped me write this explanation :+1:
Committing an update soon to reflect the changes in the readme.

Awesome, RFC reference FTW :smile: . Once the README is updated I will merge.


Hyperelastic RMT code with updated modular precision

Type of change

[x] New feature (non-breaking change which adds functionality)
[x] Something else

Scope

[x] This PR comprises a set of related changes with a common goal

If you cannot check the above box, please split your PR into multiple PRs that each have a common goal.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Tested the code on NCSA Delta and passed the full test suite and most example cases.
Provide instructions so we can reproduce.
Ran on 2 GPUs with one CPU each on an interactive node
./mfc.sh test -j 8 --max-attempt 3 -- -c delta
Test Configuration:

What computers and compilers did you use to test this:

Checklist

[ ] I have added comments for the new code
[ ] I added Doxygen docstrings to the new code
[ ] I have made corresponding changes to the documentation (docs/)
[ ] I have added regression tests to the test suite so that people can verify in the future that the feature is behaving as expected
[ ] I have added example cases in examples/ that demonstrate my new feature performing as expected.
They run to completion and demonstrate "interesting physics"
[x] I ran ./mfc.sh format before committing my code
[x] New and existing tests pass locally with my changes, including with GPU capability enabled (both NVIDIA hardware with NVHPC compilers and AMD hardware with CRAY compilers) and disabled
[x] This PR does not introduce any repeated code (it follows the DRY principle)
[x] I cannot think of a way to condense this code and reduce any introduced additional line count

If your code changes any code source files (anything in src/simulation)
To make sure the code is performing as expected on GPU devices, I have:

[x] Checked that the code compiles using NVHPC compilers
[ ] Checked that the code compiles using CRAY compilers
[x] Ran the code on either V100, A100, or H100 GPUs and ensured the new feature performed as expected (the GPU results match the CPU results)
[ ] Ran the code on MI200+ GPUs and ensure the new features performed as expected (the GPU results match the CPU results)
[ ] Enclosed the new feature via nvtx ranges so that they can be identified in profiles
[ ] Ran a Nsight Systems profile using ./mfc.sh run XXXX --gpu -t simulation --nsys, and have attached the output file (.nsys-rep) and plain text results to this PR
[ ] Ran an Omniperf profile using ./mfc.sh run XXXX --gpu -t simulation --omniperf, and have attached the output file and plain text results to this PR.
[ ] Ran my code using various numbers of different GPUs (1, 2, and 8, for example) in parallel and made sure that the results scale similarly to what happens if you run without the new code/feature

@mrodrig6 @mcarcana Hyperelasticity has no tests in the test suite... can you add them, or this somehow intentional?

The chemistry cases are failing... I'm wondering if one of our merges, we/I messed up chemistry. I suspect so. @mrodrig6

@mrodrig6 I think I fixed everything that was breaking the CI.
Two to-dos that I cannot seem to do myself:

Need new CI tests for hyperelasticity
Need to resolve about the remaining review comments I left above that haven't been resolved yet. Any questions on that just let me know.

requesting code review from @ChrisZYJ

@mrodrig6 @mcarcana Hyperelasticity has no tests in the test suite... can you add them, or this somehow intentional?

Added just now, but as examples. I need to verify the examples first.

@mrodrig6 Could you please take a look at issue #771? It would be very helpful if you could identify the error and implement a quick fix. If not, I'm happy to help revert your previous PR's changes to the hypoelasticity module - it seems that would solve the problem. (It appears your previous PR changed how finite difference coefficients are computed, but didn't add new functionality for hypoelasticity. Please correct me if I'm wrong.)

@ChrisZYJ I merged your PR if you could try to help get those features back into this one, it would be highly appreciated!

I can first help merge that PR. @mrodrig6 Could you please give me the access to edit? Thanks!

I’ve identified the issue. In the original code, hypoelastic calculations defaulted to 4th-order finite differencing regardless of input file parameters. However, the updated code adjusts the order based on the fd_order parameter.
Initially, fd_order was intended solely as a post-processing parameter for vorticity and numerical Schlieren calculations. It didn't affect the simulations, as stated in the documentation and case file comments. In the #771 case file, fd_order was set to 1. With the change in default behavior, the simulation now uses first-order finite differencing, which is less stable and led to the observed stability issues.
@sbryngelson, could you please revert my previous PR? The simulation results have always been correct, and the discrepancy came from the change in default behavior of hypoelastic calculations. Sorry for my oversight. I can update the documentation later to clarify the role of fd_order.

I’ve identified the issue. In the original code, hypoelastic calculations defaulted to 4th-order finite differencing regardless of input file parameters. However, the updated code adjusts the order based on the fd_order parameter.
Initially, fd_order was intended solely as a post-processing parameter for vorticity and numerical Schlieren calculations. It didn't affect the simulations, as stated in the documentation and case file comments. In the #771 case file, fd_order was set to 1. With the change in default behavior, the simulation now uses first-order finite differencing, which is less stable and led to the observed stability issues.
@sbryngelson, could you please revert my previous PR? The simulation results have always been correct, and the discrepancy came from the change in default behavior of hypoelastic calculations. Sorry for my oversight. I can update the documentation later to clarify the role of fd_order.

I created a PR for this: https://github.com/MFlowCode/MFC/pull/776
@ChrisZYJ can you look it over to be sure it's correct and then approve it and I will merge. Thanks!

I can first help merge that PR. @mrodrig6 Could you please give me the access to edit? Thanks!

@ChrisZYJ Yes, and I added you now.

@mrodrig6 Could you please take a look at issue #771? It would be very helpful if you could identify the error and implement a quick fix. If not, I'm happy to help revert your previous PR's changes to the hypoelasticity module - it seems that would solve the problem. (It appears your previous PR changed how finite difference coefficients are computed, but didn't add new functionality for hypoelasticity. Please correct me if I'm wrong.)

@ChrisZYJ Yes and done. I responded earlier this week to this comment. I agree with @sbryngelson that this is resolved

I’ve identified the issue. In the original code, hypoelastic calculations defaulted to 4th-order finite differencing regardless of input file parameters. However, the updated code adjusts the order based on the fd_order parameter.
Initially, fd_order was intended solely as a post-processing parameter for vorticity and numerical Schlieren calculations. It didn't affect the simulations, as stated in the documentation and case file comments. In the #771 case file, fd_order was set to 1. With the change in default behavior, the simulation now uses first-order finite differencing, which is less stable and led to the observed stability issues.
@sbryngelson, could you please revert my previous PR? The simulation results have always been correct, and the discrepancy came from the change in default behavior of hypoelastic calculations. Sorry for my oversight. I can update the documentation later to clarify the role of fd_order.

I created a PR for this: #776
@ChrisZYJ can you look it over to be sure it's correct and then approve it and I will merge. Thanks!

I agree with this causing the issue and good to know.

So if I understand correctly, this PR is waiting on tests and then it will be ready for code review? (of course I can review it earlier as well)


Corrected the spliced index in _removeFile

This is to fix an issue with the _removeFile() function in v1.2.3 with the following actual/expected behavior:
Actual Behavior:

With four items uploaded into the px-file-upload, clicking the X icon on the third removed the first in the list.
With six items uploaded, clicking the X icon on the final item in the list removed the first one.

Expected Behavior:

When multiple files are uploaded and the X icon is clicked, the file corresponding to that X icon is removed. E.g. if you click the X for the third file in the list, it removes the third file.

The fix was to get the index from e.model.__data__ when splicing the files array, so that the expected index is removed.

Thanks for the PR, Mike. I've fixed this in v1.2.5 in a slightly different way by using the Polymer API for event retargeting. Good catch.


FIX - Correction of total list count with pagination.

The situation in which the paginated list was not counted correctly has been corrected, because when there was a distinct() in the query, it did not use this parameter for the total count of items.
The option to enter the column for the correct count in paginate() has now been added.

@Fer-souzaa I'd take a look into the previous attempts before trying again:
https://github.com/laravel/framework/pulls?q=is%3Apr+pagination+distinct


sign up for Abhiti Mishra

Signed up in the sign_here.html file

Hi,
I have pushed a new sign_here html file after using vi editor.
Regards,
Abhiti
On Sun, Nov 5, 2023 at 9:50 PM Edward Ionides @.***>
wrote:

@.**** requested changes on this pull request.
Multiple lines have been changed, perhaps by using a web editor. If you
like, you can resubmit.
—
Reply to this email directly, view it on GitHub
https://github.com/ionides/810f23/pull/8#pullrequestreview-1714192409,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDYFZVYEZM5CZM5UNWDIJF3YDA66XAVCNFSM6AAAAAA66TK7DKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOMJUGE4TENBQHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


update packages

:white_check_mark: Build TheIdServer 1.0.2-fix-dependencies1-0053 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/730b1d8458 by @)


Add accent insensitive query mode

Problem
Currently there is a possibility to search a database case insensitive with QueryMode.insensitive. I would like to additionally search the database accent insensitive.
Suggested solution
Extend the QueryMode.insensitive to be also accent insensitive or add the QueryMode.accentInsensitive with the possibility to combine the mode with QueryMode.insensitive (it would probably be good to rename QueryMode.insensitive to QueryMode.caseInsensitive)

Hey @friebetill, I can understand the use case. This is unfortunately a very complex subject and our current implementation is best-effort and depends on a lot of factors.
Do you know how you would do this in SQL? If we have the SQL and potentially database collation settings, we can have a closer look.

Thanks for your answer! So far I haven't implemented anything, but I will probably follow this Stack Overflow answer and use the extension unaccent to solve it. I hope that this extension can be installed directly in Postgres and then used with the Prisma client via $queryRaw.
Of course, the approach does not work for the other databases and the first search result, e.g. for MySQL is not promising.

@friebetill If you've had any success with installing the module and using the function in a $rawQuery (and still have access to it, because it's been a while since you posted), please share here because I'm racking my brains to find out how to implement this :)

Unfortunately, I didn't try to solve the problem further because my capacity is limited.

Any updates ?

It will be very helpful to get "unaccent" from postgreSQL working without having to use query raw

Yes same thing, the whole search and fulltextsearch features of prisma become kinda useless at the moment you have non ASCII characters contents.
When you have multi language contents (japanese, chinese, ...) It become impossible to do a decent db search with default prisma tools and casing.
It's frustrating to rely on raw query just to wrap the field you search or filter lower(unaccent(title)).

Any news on this Unaccent use case?

To be able to use "unnacent" with a complex/typesafe "where", I'm using 2 queries:

The first using "queryRawUnsafe" that returning IDs
The second using these IDs to compose a native Prisma query.

Hope it helps someone :)
Does anyone have a better approach?
Example

Accent insensitive search is very important for languages like Portuguese and others.
When trying to search "João" people may use "joao" and this should be handled by prisma to avoid doing a lot of raw queries to make that work!

Any news about this?

Hey @friebetill, I can understand the use case. This is unfortunately a very complex subject and our current implementation is best-effort and depends on a lot of factors.
Do you know how you would do this in SQL? If we have the SQL and potentially database collation settings, we can have a closer look.

Hello there, I'm also in need of searching without having to think about accent.
Unfortunatly, I didn't have the chance yet to deep dive into the code of Prisma. So maybe what I am about to say is retarded :)
If it's complicated to handle it for all languages, maybe we could settle for a solution where Prisma provides a hook called when the unaccent parameter is set? The objective of this function would be to locally modify the query and apply PSQL functions like unaccent ?
It would prevent the use of raw query for the time being, and once the feature is fully operational, the hook can be ignored.

Ahhhh, remembering how I love modern ORMs...
"Prisma is a next-generation Node.js and TypeScript ORM that unlocks a new level of developer experience"

with queryraw you can do it like this
const rows = await prisma.$queryRaw       select *       from personas        where translate(nombres,'áéíóúÁÉÍÓÚäëïöüÄËÏÖÜ','aeiouAEIOUaeiouAEIOU') ILIKE '%' || translate(${search}, 'áéíóúÁÉÍÓÚäëïöüÄËÏÖÜ','aeiouAEIOUaeiouAEIOU') || '%'        or translate(apellidos,'áéíóúÁÉÍÓÚäëïöüÄËÏÖÜ','aeiouAEIOUaeiouAEIOU') ILIKE '%' || translate(${search}, 'áéíóúÁÉÍÓÚäëïöüÄËÏÖÜ','aeiouAEIOUaeiouAEIOU') || '%' limit 30;

Any news? Let's speed up this issue!!

Hey! I think this is a must for latin languages. A proper search can't be done without this.


Cannot change Camera State on iPhone 4s

I cannot use the init method with cameraState or change the camreaState value at all on a 4s device.
Ways to reproduce:
1.
- (void)viewDidLoad {
    [super viewDidLoad];
    _session = [[VCSimpleSession alloc] initWithVideoSize:CGSizeMake(480, 480) frameRate:30 bitrate:1000000 useInterfaceOrientation:YES cameraState:VCCameraStateFront];
    _session.previewView.frame = self.previewView.bounds;
    _session.delegate = self;
}

Preview view shows black screen, nothing else.

- (void)viewDidLoad {
    [super viewDidLoad];
    _session = [[VCSimpleSession alloc] initWithVideoSize:CGSizeMake(480, 480) frameRate:30 bitrate:1000000 useInterfaceOrientation:YES];

    [self.previewView addSubview:_session.previewView];
    _session.previewView.frame = self.previewView.bounds;
    _session.delegate = self;
}

// ...

- (IBAction)changeCamera:(id)sender
{
    switch (_session.cameraState) {
        case VCCameraStateFront:
            _session.cameraState = VCCameraStateBack;
            break;

        case VCCameraStateBack:
            _session.cameraState = VCCameraStateFront;
            break;

        default:
            break;
    }
}

Session starts fine using the front facing camera.  When I try to switch the camera using the changeCamera method above, the UI freezes.
Device info:
iPhone 4s 16gb model running 8.1.2 (12B440)
Let me know if you need any more information.
Thanks.

Any updates on this?  This is also reproducible in the sample app.

I haven't had the opportunity to give it a test since I don't have any device like the 4s.  Can you run the program in the debugger, and when it freezes press the Pause button and then type "bt all" in the lldb console and paste the resulting stack trace to a gist?

Here's the backtrace:
https://gist.github.com/JALsnipe/1577094029661130eb3a
I also found another 4s running 8.0.2.  Surprisingly, the camera switched fine on that device:
https://gist.github.com/JALsnipe/f6215ccbc54e4dd37210
I have a third 4s device I am upgrading to 8.1.2 now to test again.  I'm wondering if this weird bug is device-specific, or versions specific.
Thanks again for all of your help.

Confirming that this may be a device-specific issue.  When I tried the sample project on a third 4s with 8.1.2, the camera was switching fine:
https://gist.github.com/JALsnipe/184d10fbf718d8910d40

Hm, try the latest commit I made - I changed the toggle method a bit.

Doesn't appear to have changed anything:
https://gist.github.com/JALsnipe/19bf293ab2d5e7de6686
I don't know why this could be happening.  Both of the device's cameras function fine.  At least it isn't a complete blocker since I haven't been able to produce it on other 4s devices.

Going to close this, pretty sure this was a device-specific defect.  If anyone has a similar issue I'd be happy to reopen and help them.


Upgrade AutoCommon 1.2.1 -> 1.2.2

This PR contains the following updates:

Package
Type
Update
Change

AutoCommon (source)
compile
patch
1.2.1 -> 1.2.2

[ ] If you want to rebase/retry this PR, check this box

Warning
Renovate's suggested commit message is being replaced with improved initial commits to enable automerging. As a side effect, these suggested commit messages might have changed. Consider comparing the initial commit message and this message to determine the most suitable one. Please leave feedback in #sys-renovate.

Suggested commit message:
Upgrade AutoCommon 1.2.1 -> 1.2.2

See:
- https://github.com/google/auto/releases/tag/auto-common-1.2.2
- https://github.com/google/auto/compare/auto-common-1.2.1...auto-common-1.2.2

Willl retrigger;

Caused by: org.apache.maven.enforcer.rule.api.EnforcerRuleError: Unable to process dependency com.github.PicnicSupermarket.error-prone:error_prone_core:jar:v2.19.1-picnic-1:provided due to null

@Stephan202 the build was successful, it's ready for your eyes :).


”Recea”, ”Șăușa”, ”Vidrasău” are listed in the drop-down menu

Description:
When user writes ”Recea”, ”Șăușa”, ”Vidrasău” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Recea"

Expected results
The location appears in the drop-down menu as "Sat Recea, MUREȘ (UNGHENI)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Șăușa"

Expected results
The location appears in the drop-down menu as "Sat Șăușa, MUREȘ (UNGHENI)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Vidrasău"

Expected results
The location appears in the drop-down menu as "Sat Vidrasău, MUREȘ (UNGHENI)".


Doesn't compile simple_game for android

C:\Users\Mercury\raylib\raylib\templates\simple_game>C:\MinGW\bin\mingw32-make PLATFORM=PLATFORM_ANDROID

mingw32-make -f Makefile.Android
mingw32-make[1]: Entering directory 'C:/Users/Mercury/raylib/raylib/templates/si
mple_game'
if not exist android.simple_game mkdir android.simple_game
/usr/bin/sh: -c: line 1: syntax error: unexpected end of file
Makefile.Android:142: recipe for target 'create_temp_project_dirs' failed
mingw32-make[1]: *** [create_temp_project_dirs] Error 1
mingw32-make[1]: Leaving directory 'C:/Users/Mercury/raylib/raylib/templates/sim
ple_game'
Makefile:293: recipe for target 'all' failed
mingw32-make: *** [all] Error 2

@protondev Are you launching Makefile from a DOS window?

Yes.
6 Mart 2019 Çarşamba tarihinde Ray<EMAIL_ADDRESS>yazdı:

@protondev https://github.com/protondev Are you launching Makefile from
a DOS window?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/raysan5/raylib/issues/774#issuecomment-470062831, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AoERK6fdGrdDaIN0PU79AYEXKrRITbIVks5vT55LgaJpZM4bbQF4
.

Reviewed build paths in commit https://github.com/raysan5/raylib/commit/0bbf857b00ab111749aa7ae22d942108ef06455f
Just tested and seems to work ok, just make sure to build raylib Android library before building game and library libraylib.a is generated in raylib/src directory.

I had a similar issue, it was because mingw32-make used bash instead of cmd. SHELL=cmd helped.


Clipboard Auto Sync

Is it possible to enable auto syncronization of the remote clipboard with the local clipboard?
It is a bit of a pain to have to paste into the clipboard widget and then paste again inside the VNC session and to remember to do so.

The local and remote clipboards are synchronized.
Are you suggesting that a "ctrl+v" into the clipboard widget automatically performs a "ctrl+v" into the remove app ?

I have to use the clipboard widget in the side panel in order to transfer to/from the app. I can't copy something on my host PC and paste it directly into the app or vice versa.

I can't copy something on my host PC and paste it directly into the app or vice versa.

To my knowledge this is not something that can be done: for security reason, a web page cannot freely access the clipboard without user intervention.

Would ctrl+v or ctrl+c not be intervention?
I swear I have used other HTML VNC like apps before that accomplish this, I know other apps such as code-server require HTTPS for the clipboard to work.
I'll have to see if I can find any examples, it's been a while.

Looks like it is not supported by noVNC yet, see this PR: https://github.com/novnc/noVNC/pull/1347

Thanks for the pointer!  I will follow this thread closely.


Request: Add Issue Templates

Feature Request
Description:
I would like to suggest adding issue templates to the .github folder in the project repository. Issue templates can help streamline the issue creation process by providing a standardized format for bug reports and feature requests.
Benefits:

Improve issue clarity and completeness.
Encourage users to provide necessary information.
Save time for both contributors and maintainers.

Suggested Changes:
Please consider adding the following two templates to the .github folder:

bug_report.md for reporting bugs.
feature_request.md for requesting new features.

Example Usage:
By implementing these templates, contributors can select the appropriate template when creating an issue, ensuring that relevant details are included, making it easier for maintainers to understand and address the issue.
Contribution:
I am willing to contribute to this request by submitting a pull request (PR) to add the necessary issue templates. Please let me know if you would like me to proceed with this implementation.
I believe this enhancement would benefit the project and its community. Thank you for considering this request.

@Yash-1511 sounds like it shows up in the issue creating UI so that contributors can easily select options. Sounds good to me, interested to see a PR!

Contributor

Yes @joshbickett it shows up some instruction or pre templates for creating bug issues, feature request issues or pull request so it can be easy for both contributors and authors to understand it clearly. I have created PR #121 just check this out and give me your feedback.


Best practice updates for packaging

Our packaging strategy for the template used setup.cfg and was fairly out of date. This pull request brings us in line with more modern practices for packaging. I did the following:

Put python package in a src/ folder
Replaced setup.cfg completely with pyproject.toml
Added script entry points that are compatible with pyproject.toml
Updated README to describe the new approach
Python versions changed from 3.9 and 3.10 to 3.10 and 3.11

Thanks for the suggestions, @tnmygrwl! I really like them. I'm going to merge this pull request in, just to close it out. Then I'll turn most your suggestions into issues/tasks, so they can be addressed at a later date. I don't think any of these need to be done before DS4CG 2024, but it would be nice if we can pick a few of them off.
Let me address them one by one for now:

1. Automated Code Formatting: Code formatters like [Black](https://black.readthedocs.io/en/stable/) or [Ruff](https://github.com/astral-sh/ruff) can help maintain a consistent code style across the project, making it easier to read and understand.

I like black a lot and it's already included in the developer dependencies for this repo. Maybe we just need info/links in the README on how to use it or set it up in an IDE?

2. Type Checking: [mypy](https://mypy-lang.org/) or similar tools for static type checking. This can help catch certain types of errors before runtime and can make the code easier to understand.

Yep, makes sense. I'll make a task.

3. CI: the template already mentions using `pytest`, we could also include a basic CI setup using GitHub Actions or Travis CI. This would automatically run tests on each commit or pull request, helping to catch issues early.

There is already a GitHub action set up for build and test in this repo (from earlier, not in this pull request).

4. Code Coverage: Along with testing, we could add a tool like [Coverage.py](https://coverage.readthedocs.io/en/7.3.3/) to measure code coverage. I have also used `pytest-cov` in a project. Although maximizing coverage is a flawed metric to optimize for, it's still helpful and improves test writing.

Makes sense and should be very low effort to add.

5. Pre-commit Hooks: Consider using a tool like pre-commit to automatically run checks before each commit. This can help catch issues early and ensure only clean, passing code is committed.

I'm not sure about this one, because it adds more complexity to version control, which most students are still learning when they start working with us. I'd rather have everything as GitHub actions, to avoid local problems with git and for easier transparency on what checks are run.

6. Dependency Management: Consider using a tool like Poetry for managing project dependencies. This can help ensure consistent environments and make it easier to manage project dependencies. Poetry uses `pyproject.toml` and can be used to build and distribute projects too.

I have poetry listed as a suggestion in the README right now. I wouldn't mind switching the whole repo over and making poetry a best practice for us, but I don't consider it really high priority right now.

7. Security Checks: Consider integrating a tool like [Bandit](https://bandit.readthedocs.io/en/latest/) to find common security issues in your Python code. Underrated tool for security-critical apps.

Sure, this seems useful. I don't think we have many projects that need it though, so it's low priority in my mind.

8. Code Complexity Measurement: I've also seen tools like [Radon](https://radon.readthedocs.io/en/latest/intro.html) be used to compute various metrics about the code, such as its complexity. This can help maintainability and for scoping.

I'm not very familiar with these types of measures and haven't used them before, and would prefer to try them on a staff project first to learn more. For now, I'd rather deal with code complexity and maintainability in code reviews, so we can give targeted feedback. I'm also worried these metrics might be confusing or frustrating for students who are less experienced. We can talk more about this at a 1:1.

9. Docstrings: While the project uses Sphinx for documentation, it could be beneficial to enforce a consistent docstring style (like Google or NumPy) using a tool like pydocstyle.

Sure, I'll make a task for it.


Fix bladedrone logo shadow

The shadow gets clipped by the boundaries of the PNG. Make the shadow smaller or the PNG dimensions larger.

fixed in: 2681e6cfbec09206c99cb4a68dd33744e631af49


Update to Collective and open research in planetary sciences...

I changed the project title from "Collective and open research in planetary sciences, with a focus on climate change, space exploration, and astrobiology" to "Collective and Open Research in Climate Science" and fixed the broken links

Thanks @varlottaang for the changes!


Dynamic GeoCRS Tiles

@AndrewAnnex Here is an example generating TMSs for planetary spherical GeoCRS dynamically. I agree, that this is going to have naming issues, but I think the logic is sound to bulk generate the needed json files.
I had to add a requests dependency to the lib, but that's trivial.
Thoughts?
https://gist.github.com/jlaura/75c6819f286d13b94458949b9df0c63d

hey @jlaura, adding requests as a dependency is no problem really, but could also be solved by running wget in the github action. I like the idea of keeping this project around to run the workflow of generating the tms so that there is a single publicly available source for these files. I can easily adapt my code to work more like your gist to work from text files, were you planning to open a PR? If not is it okay for me to adapt it to this codebase?
If you open a PR that would allow some collaborative editing, I could also open a PR but i'd have to add you to the repo, which I'm also cool with, just thinking through the options. let me know what makes the most sense to you of those

@AndrewAnnex I can get a PR open that we can both work on. I wanted to see if this was inline with your thinking. I can also email with a TiTiler endpoint that we can deploy these too and test. I'd like to keep that private for now.

Okay sounds good to me- Dr. Andrew AnnexOn Dec 7, 2022, at 10:00 AM, jlaura @.***> wrote:
@AndrewAnnex I can get a PR open that we can both work on. I wanted to see if this was inline with your thinking. I can also email with a TiTiler endpoint that we can deploy these too and test. I'd like to keep that private for now.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>


Add AVC vref for Solver Engines

https://github.com/KSP-CKAN/CKAN-meta/compare/015c586db56e...ece58f800ef7 added SolverEngines v1.10 which has a working .version file. Huzzah!


WIP Itemvalue misc fixes

includes multiple small fixes to ItemValue functionality:
2586167 Fixed datavalidation to only check for requirements of used regions
63605b3 Made it so for sure that value name get lowercased (check items.py)
2b97f9d Renamed world.item_values_cache to world.itemvalue_rule_cache to stop potential confusion with world.item_values
5e64593 Stopped PreFill Datavalidation of ItemValue from reseting the world.item_values cache when running get_items_with_value so hook user can run it first then edit it to modify an item value for dev reasons
for some reason the diff of  commit 63605b3 failed the only line changed in datavalidation is 249 -> 278

since the first commit diff failed I might try to cherry pick the other commit in a new branch and then make a new pr idk

Would love to get the item value lower fixes (https://github.com/ManualForArchipelago/Manual/commit/63605b3b1c34e521d52d77b0d297301ab7296f58) by themselves in a PR. That one's easy to digest and get through, and would fix a common issue people have with ItemValue in general.
I have to ask, though... why do they have to be lowercase?

Would love to get the item value lower fixes (63605b3) by themselves in a PR. That one's easy to digest and get through, and would fix a common issue people have with ItemValue in general.
I have to ask, though... why do they have to be lowercase?
@FuzzyGamesOn
Since ItemValue has been a thing Id alway .lower() the value names in some situation (like the item groups) but I forgot to do it in some places and that lead to some issues (like data validation)
all this pr above does is to make it so the actually value get lower even if the dev write Star instead of star
I could go and remove that if you prefer
ill probably redo this pr anyway for the weird diff issue

Redid this pr without the change that bugged the diff see #104


Add 'jerry_job_queue_is_empty' function to the API.

This API function could be useful on systems which use eventloop and
real asyncron calls, like IoT.js.
JerryScript-DCO-1.0-Signed-off-by: László Langó<EMAIL_ADDRESS>

@LaszloLango Could you give a quick example that in which cases we need call this API?

@jiangzidong https://github.com/Samsung/iotjs/issues/1292

So it is more like a debug tool, right?

No, it is not a debug tool.

Why it is not enough to simply run the jobqueue? I suspect it is nop if the queue is empty.

sorry I got confused.
the logic about the jobqueue and eventqueue should be
while(if there is pending event like timer, fs, network, ...)
{
   wait the event triggered
   call the corresponding event handler (usually will call js callback, but its ok to call a pure native handler)
   call jerry_run_all_enqueued_jobs, which will make sure all the enqueued job are executed in sequence
}

So I don't know when to use the jerry_job_queue_is_empty.
Back to the iotjs issue, the workaround is counting the pendingPromiseObj, note that the pendingPromiseObj will not be added into the jobqueue, only the resolved promise will be added into the jobqueue. For example. if the async readfile takes 5s to finish reading, then during the 5s, we have the pendingPromiseObj, but the jobqueue is still empty

@jiangzidong no worries. I found a better fix this. I don't need this new API anymore :) Sorry for the inconvenience.


Get coefficients when using .over

This is such a great tool!
I have one question, not sure if its a feature request or I've just failed to understand something.
I'd like to get the coefficients when doing regression by group (so I can apply them to new data).
Taking your example I'd like to do:
ols = pls.compute_least_squares(pl.col("y"),pl.col("x1"),mode="coefficients").over("group")
df.select(ols)

This works find without over and each row is a coefficient. With over this would need to be a long dataframe with a column showing which group and which coefficient it belongs to.
It looks like the output in this case would need to be a pl.Struct with group, variable name and value fields.
Any thoughts on this?

Hi @braaannigan, thanks!
No, you did not misunderstand anything at all, what you describe is a perfectly valid feature request (and better default behaviour in general). Right now mode=predictions will broadcast to any shape but you are right in “coefficients” not doing that correctly right now.
I think the moving window models (RLS, Rolling) will have their coefficients broadcast to any shape correctly right now because of the way their coefficients are represented (as List[f32]).
If we change the default “coefficients” behaviour to return a list (or a struct as you noted) the result should broadcast nicely too. (.with_columns would repeat the coefficients across all rows to match original data shape and .select will produce a long frame per group as you expected it should be for your usecase).
let me think precise mechanics a little more but should definitely be able to do something that will get you what you want! Thanks for raising this.

Hi @braaannigan
The latest version basically does exactly the behaviour you describe (with both the struct creation and subsequent out-of-sample prediction of a struct coefficients column onto test features done efficiently in rust); so closing as completed pretty much in line with your comment (thanks!). However, please drop a line if anything doesn't work as expected and happy to take another look
To recap, your example would become:
df = _make_data(n_groups=10)  # your train data
df_coef = df.select(
        "group",
        pls.compute_least_squares(pl.col("y"), pl.col("x1"), pl.col("x2"), mode="coefficients")
        .over("group")
        .alias("coefficients")
        )

>>> df_coef.head()
shape: (5, 2)
┌───────┬─────────────────────┐
│ group ┆ coefficients        │
│ ---   ┆ ---                 │
│ i64   ┆ struct[2]           │
╞═══════╪═════════════════════╡
│ 2     ┆ {0.997433,1.00141}  │
│ 7     ┆ {0.997332,0.99887}  │
│ 5     ┆ {1.000659,1.002979} │
│ 1     ┆ {0.998089,1.002442} │
│ 6     ┆ {0.999837,0.996519} │
└───────┴─────────────────────┘

There will be as many rows of coefficients as samples in your dataframe because the extension packages will broadcast here (and if you take .unique(), you'll get as many unique coefficient rows as groups in this particular example).
The field names of the struct are, as you expect, the names of the features model was trained on - so you can "unnest" to get:
>>> df_coef.unnest("coefficients").head()
shape: (5, 3)
┌───────┬──────────┬──────────┐
│ group ┆ x1       ┆ x2       │
│ ---   ┆ ---      ┆ ---      │
│ i64   ┆ f32      ┆ f32      │
╞═══════╪══════════╪══════════╡
│ 2     ┆ 0.997433 ┆ 1.00141  │
│ 7     ┆ 0.997332 ┆ 0.99887  │
│ 5     ┆ 1.000659 ┆ 1.002979 │
│ 1     ┆ 0.998089 ┆ 1.002442 │
│ 6     ┆ 0.999837 ┆ 0.996519 │
└───────┴──────────┴──────────┘

To predict on some test data, you would simply need to align your train coefficients with test data and run pls.predict(...) (or  equivalentlypl.col("coefficients").least_squares.predict(...)):
df_test = _make_data(n_groups=10)  # your test data
df_test = (
    df_test.join(df_coef.unique(), on="group")  # or join on whatever
    .with_columns(pls.predict(pl.col("coefficients"), pl.col("x1"), pl.col("x2"))
                  .alias("test_predictions"))
)
>>> df_test.head()
shape: (5, 6)
┌───────────┬───────────┬───────────┬───────┬─────────────────────┬──────────────────┐
│ y         ┆ x1        ┆ x2        ┆ group ┆ coefficients        ┆ test_predictions │
│ ---       ┆ ---       ┆ ---       ┆ ---   ┆ ---                 ┆ ---              │
│ f32       ┆ f32       ┆ f32       ┆ i64   ┆ struct[2]           ┆ f32              │
╞═══════════╪═══════════╪═══════════╪═══════╪═════════════════════╪══════════════════╡
│ 0.025985  ┆ 0.12573   ┆ -0.132105 ┆ 2     ┆ {0.997433,1.00141}  ┆ -0.006884        │
│ 0.857969  ┆ 0.640423  ┆ 0.1049    ┆ 7     ┆ {0.997332,0.99887}  ┆ 0.743496         │
│ -0.160373 ┆ -0.535669 ┆ 0.361595  ┆ 5     ┆ {1.000659,1.002979} ┆ -0.17335         │
│ 2.308727  ┆ 1.304     ┆ 0.947081  ┆ 1     ┆ {0.998089,1.002442} ┆ 2.250902         │
│ -1.944108 ┆ -0.703735 ┆ -1.265422 ┆ 6     ┆ {0.999837,0.996519} ┆ -1.964638        │
└───────────┴───────────┴───────────┴───────┴─────────────────────┴──────────────────┘

It should be pretty efficient and several additional / edge cases are now in the tests
Finally, your train data may have nulls in it, so in the latest version, you can pass null_policy="zero", "drop", etc. and the library will handle them for you based on your choice prior to fitting.


Force bulletin's dt_assinatura with today's date

Force bulletin's dt_assinatura with today's date
File:
medirepoWeb/src/pages/hospitals/app/components/BulletinAdd.js

Fixed


Fix 2 typos in code that has zero test coverage

Use \, not / to continue one line onto the next.
Can't call #red on the string, only on the Rainbow instance.

Introduced by accident in #2205.
cc @daviddavis

:+1: good catch. Thanks!


Modify Freecen2 Place Cache to update the data_present flag based on confirming that there are files present

Currently the flag is never checked

Have modified the rake task foo:check_and_refresh_freecen2_places_cache to  clear and reset the data present flag from scratch.
It rebuilds the place list for a county in the search

Completed and deployed to test3


BEAM kernel count number is wrong

beam2  : 16 31 31 16 2 3 2 4 3 2        :   817.92 us  <  hc     : 4 31 31 32 4 3 3 4 2 2         :  1000.83 us
*** GPU        9 r_16_31_31_16_2_3_2_4_3_2              arg   3 mem  0.87 GB tm   1244.89us/     4.99ms (  113.83 GFLOPS,   24.03 GB/s)
   0.00s:                 from   1 ->   1 actions  512   32   32   32   12
   4.62s:      7940.00 us from  30 ->  30 actions  512    2   32   32   16   12
  13.97s:      3760.83 us from  59 ->  59 actions  512    2   32    8   16   12    4
  23.99s:      2318.33 us from  56 ->  56 actions  512    2   32   16   12    4    8
  35.39s:      1741.56 us from  47 ->  47 actions  512   32    4   16    2   12    4    2
  43.81s:      1741.56 us from  41 ->  41 actions  512   32    4   16    2   12    4    2
beam2  : 512 32 4 16 2 12 4 2           :  1778.44 us  <  hc     : 512 32 2 16 8 4 12             :  3710.63 us
*** GPU       16 r_512_32_4_16_2_12_4_2                 arg   3 mem  0.96 GB tm   1835.73us/     6.83ms (  274.18 GFLOPS,   49.42 GB/s)

Kernel number jumps from 9 to 16, it should only go up by one (like from 9 to 10)

The problem is with this realize, will make the pr after my final exam (in 11mins).


Not supports IE browsers?

The PaperRipple examples in Codepen is good, but when I use npm install paper-ripple to install latest version 0.2.0, seems  not work at IE 11

@rayliao, will check it later when get back to home. Thank you.

@virtyaluk Hi, Do you have any solution yet? :)

@rayliao
Try this. It woks in IE 10+.
.paper-ripple--round .paper-ripple__background, .paper-ripple--round .paper-ripple__waves {     border-radius: inherit; } .paper-ripple--round .paper-ripple__wave-container {     overflow: visible; }


Add array_average Presto function

Implementing array_average as a simple function. Based on https://github.com/facebookincubator/velox/pull/2434.

@aditi-pandit Could you review this one as well? It is basically the same version as the original PR, I just slightly updated comments and the docs.

The CI error does not seem related to the code, I am going to retrigger the build.

@mbasmanova I would appreciate it if you could review this PR. Thank you.

Thank you for the review, I have addressed the feedback. Please take another look.

Thanks for review. I addressed the comments.


Your new paper: "Towards Noise-resistant Object Detection with Noisy Annotations"

Dear @LiJunnan1992 ,
I am very interested in your research, especially your new paper . Are you going to release its source code so that we can reproduce the results?

Hi, thanks for your interest! We will release code for that paper upon acceptance.

Thanks for your time.

Dear @LiJunnan1992 ,
I am very interested in your research, especially your new paper . Are you going to release its source code so that we can reproduce the results?


draft adr for on-call schedule

TBD

Warnings

:warning:

Please include the Jira ID at the start of the title with the format MB-123 or [MB-123]

:warning:
Please include a description of your PR changes.

  Generated by :no_entry_sign: dangerJS against cf3082f509a5088ad8ad4e037ed1b06036f964c7

I like where this doc is going. Make sure to include a section on how this will affect training of folks on how to do on-call (especially on-boarding).

Yep, next steps are to talk to a couple of people and figure out plan for selected alternative.

Closing this and replacing with Process Decision Record


updated task-set-5.md

Completed 5th topic

@heman1 Please update the link with the publication's link after the post is published

i am not able to submit the article as i am not a contributor to that publication.

I have changed the medium blog link.


Delete term groups that do not exist anymore

Manual cleanup via deleting all content is supported via the --refresh option to the sync and populate  management commands, but we may still want a more surgical mechanism.


split1 methods use numerical values which aren't traditional in Rust naming conventions

Hello! Really excited to see that Facebook is contributing to the Rust ecosystem and surprised there isn't much activity here.
If this is intentional feel free to ignore me, just browsing through the code and saw this one nit. Should the methods that split at the first index of a string use '1' in their name (e.g split1(), split1_opt()) instead of split_first() or split_start()?

The reason was that split gives you all the splits, splitn gives you the first N splits, so the logical name for the first 1 split is to substitute 1 for n and get split1. I appreciate it's not common in Rust functions, but in some ways you can see splitn as the thing that broke that first (since n in this context is a placeholder for a number). Certainly split_first would be a perfectly good name, although given that it already has a name, I'm not sure it's sufficiently better to justify changing it. That said, our hope is that eventually these functions end up in Rust proper - for example split1_opt is now in Rust as split_once - so if they change name when moving over that seems perfectly fine.

Ah I understand, I'll go ahead and mark this closed then.
Thanks for the explanation!


Complex STFT transform from spectrogram

We offer a layer that applies complex STFT without the power/normalization done in spectrogram. However, we do not want to overload the name "STFT" existing in pytorch, as done in the original proposal in the proposal from #285.
@ksanjeevan -- thoughts?
CC @ksanjeevan @keunwoochoi @cpuhrsch

@vincentqb Yeah that would work! So I'm not seeing anything on Spectrogram that checks that power is int or anything like that so just with this change of the functional we could have a complex stft transform correct?

@vincentqb Yeah that would work! So I'm not seeing anything on Spectrogram that checks that power is int or anything like that so just with this change of the functional we could have a complex stft transform correct?

Right, there should only be a check for none to disable the power computation


Mapping Rule generation fails for large White Rabbit Scan reports

Timeout occurs for large scan reports, meaning manual intervention is necessary.
Examples in comments

This is being tackled by moving this task to an Azure Function. If we implement the setting of PersonID and Date Event from the Data Dictionary, then this would all happen at the time of upload rather than once all Person IDs and Date Events are set, removing a confusing and at times painful process for users.


Numba types

Python complained when calling function coreICSA(...) from this line, since the types of input parameters were unexpected.
We solved by doing the following conversions:
startTime = numpy.int32(startTime)
arrayCC = arrayCC.astype(numpy.int64)

Ok!


Persist files and scene state in Excalidraw plugin

When adding images to the Excalidraw scene, the files were not persisted
When exporting the scene, it was losing some of the scene data, like background color.

Before:
https://user-images.githubusercontent.com/7893468/231016887-7a68c03b-341b-4156-b78c-414b828893b3.mp4
After:
https://user-images.githubusercontent.com/7893468/231016948-61eae2ab-d813-4de5-b588-e5bd7732fbdd.mp4

Sorry, I didn't have a chance to look too closely - is this just storing the image data in the actual excalidraw state? Is it base64 encoded? That might lead to some absolutely massive states with large images the 33% penalty from b64.
Is there support for storing just a reference to the image? Does this PR allow that?

Sorry, I didn't have a chance to look too closely - is this just storing the image data in the actual excalidraw state? Is it base64 encoded? That might lead to some absolutely massive states with large images + the 33% penalty from b64.
Is there support for storing just a reference to the image? Does this PR allow that?

Hey, @acywatson, you are spot on, if someone adds an image, it stores the image as a base64 encoded string in the scene state. Can it lead to massive states and thus a heavy document? Yes, absolutely. That said, I'd like to believe, users are conscious of what adding too many images this way means and I'd take a fully working Excalidraw component with all features working that they can disable (eg. by hiding the "add image" button inside the component), rather than have it miss functionality. I don't have a strong opinion on it, if you want to hide the button in the playground, happy to do it as well.

Sorry, I didn't have a chance to look too closely - is this just storing the image data in the actual excalidraw state? Is it base64 encoded? That might lead to some absolutely massive states with large images + the 33% penalty from b64.
Is there support for storing just a reference to the image? Does this PR allow that?

Hey, @acywatson, you are spot on, if someone adds an image, it stores the image as a base64 encoded string in the scene state. Can it lead to massive states and thus a heavy document? Yes, absolutely. That said, I'd like to believe, users are conscious of what adding too many images this way means and I'd take a fully working Excalidraw component with all features working that they can disable (eg. by hiding the "add image" button inside the component), rather than have it miss functionality. I don't have a strong opinion on it, if you want to hide the button in the playground, happy to do it as well.

No, I think we're good - I discussed offline with @zurfyx. it's just playground code, but I had a feeling best practice might be somehow adding a ref to the image and loading it async from storage, I just don't know how that's done in excalidraw. It's a problem for another day - this is good work and thank you for doing it :)


Stop doing two transposes for each transform

This looks pretty simple. The only functional changes that I see having to be done is removing the transposes and changing the scan order arrays.
I might have missed something, though.

This looks pretty simple. The only functional changes that I see having to be done is removing the transposes and changing the scan order arrays.
I might have missed something, though.

I don't see where we need to use scan order arrays when doing 2-d transforms?
Given the process in video compression that a step of scanning coefficients is happening only after transform is applied.
Also the reason for doing two transposes is, in one transform stage of of two stages of transforms (which dimension is entirely decided by whether the input elements are stored in a row, i.e. consecutive memory) requires to store input and its output in a consecutive memory for memory access (yes, CPU caches) efficiency reason.
If you start horizontal transform first, that is 1st stage and does not need transform.
Then in the 2nd stage, because each input is column, we have to transpose each column into row. Then apply 1d transforms there, and transpose again the result in order to have column vectors/outputs.

The scanning I'm talking about is happening here. https://github.com/xiph/rav1e/blob/master/src/context.rs#L3974
I want to change the scan order used here, so that we don't need to do the second transpose. I'm a confused by your mention of memory efficiency, since I don't see any code requiring sequential accesses that have to be across specifically columns or specifically rows.

Yeah, I think if you have given further context of "during coefficient coding" in the title of this PR, I haven't left unrelavant comment.
Like, "When coefficients are coded", Stop doing two transposes for each transform
Or, Stop doing two transposes for each transform "during coefficient coding"

Isn't this done before?

Closed by #1955


Add missing assembly metadata

CLSCompliant should be set to true in all except System.Runtime.CompilerServices.Unsafe.
IsTrimmable should be set in System.Runtime.CompilerServices.Unsafe but only on frameworks compatible with $(NetMinimum).
All assemblies should have AssemblyDefaultAlias, AssemblyDescription, PreferInbox, Serviceable and .NETFrameworkAssembly.

I confirmed with ILSpy that all tests DLLs, S.R.CompilerServices.Unsafe and Microsoft.IO.Redist are not CLSCompliant.
All the other DLLs from the migrated assemblies have the CLSCompliant attribute explicitly set to true in the metadata.

Just for my own peace of mind, I'll post screenshot comparisons of all the assemblies metadata before and after. Give me some time to collect them.

@Romfos see https://github.com/dotnet/maintenance-packages/blob/9bade88c9d87659bf1a55c756028d4c6537f9b4b/src/System.Buffers/src/System.Buffers.csproj#L4
System.Buffers got moved inbox - into the framework - with .NET Core 2.1 and hence we are adding a placeholder file to the package to indicate that.

This is very close to being done, just want to close a few questions.

I made a full comparison of each DLL before and after.
I noticed we're still missing in some of them the following metadata:

[module: NullablePublicOnly(false)] - Only in a few cases: M.Bcl.H. (netstandard2.0), M.IO.R. (net462), S.N.WS.WSP. (all)
[assembly: AssemblyMetadata("IsTrimmable", "True")] - Only in a couple of cases: S.R.CS.U. and M.IO.R.
[assembly: AssemblyDescription("When the csproj itself specified this already.")] - Only one case: M.IO.R.
[assembly: AssemblyMetadata(".NETFrameworkAssembly", "")] - We don't need to add it per this comment.

I also noticed that in the new versions, these properties are getting added that we did not use to have before:

[module: RefSafetyRules(11)]
[assembly: TargetFramework("<framework name>,Version=<version>", FrameworkDisplayName = "<display name>")]
[assembly: DefaultDllImportSearchPaths(/*Could not decode attribute arguments.*/)] - Suggested by Viktor in chat
[assembly: AssemblyMetadata("RepositoryUrl", "https://github.com/repo/url")]
[assembly: AssemblyConfiguration("Release")]

And these are metadata attributes that only some old DLLs used to have, and the new ones all have them set:

[assembly: NeutralResourcesLanguage("en-US")]
[assembly: SecurityPermission(8, SkipVerification = true)]
[assembly: SecurityPermission(8, SkipVerification = true)]


Sometimes Browser stays open

Hi,
using selenoid on Windows with grr (if that matters). Running out tests, sometimes the browser doesn't close on a windows node and selenoid last logs are: [NEW REQUEST].
For the example below, I looked at the machine at 10:00 o'clock and the last [NEW REQUEST] ist from 06:43.
We are starting selenoid like this:
selenoid_windows_386.exe -conf browsers.json -disable-docker -enable-file-upload -timeout 120s -session-attempt-timeout 120s -session-delete-timeout 120s -limit 1 -retry-count 10
Maybe driver.quit() wasnt called under some circumstance, but I would assume, that selenoid has to timeout in such a case. Any ideas?

Selenoid timeouts session only if no new http requests come to browser. Is
there something like implicit waits in your tests? Also, there is project:
https://github.com/kybu/headless-selenium-for-win
it uses native windows api to start and stop ie driver, it may be more
reliably than using go api. Try to configure selenoid like this:
{
"internet explorer": {
"default": "11",
"versions": {
"11": {
"image": ["./headless_ie_selenium.exe"]
}
}
}
It will start ie driver on virtual desktop, and will use windows jobs to
stop process subtree.
2017-09-15 11:34 GMT+03:00 Markus Krüger<EMAIL_ADDRESS>
Hi,
using selenoid on Windows with grr (if that matters). Running out tests,
sometimes the browser doesn't close on a windows node and selenoid last
logs are: [NEW REQUEST].
For the example below, I looked at the machine at 10:00 o'clock and the
last [NEW REQUEST] ist from 06:43.
We are starting selenoid like this:
selenoid_windows_386.exe -conf browsers.json -disable-docker
-enable-file-upload -timeout 120s -session-attempt-timeout 120s
-session-delete-timeout 120s -limit 1 -retry-count 10
Maybe driver.quit() wasnt called under some circumstance, but I would
assume, that selenoid has to timeout in such a case. Any ideas?
[image: selenoid_console]
https://user-images.githubusercontent.com/14224592/30473471-defee6f8-9a00-11e7-8cc7-58904996ff61.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aerokube/selenoid/issues/216, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABWNbUWIMax2hbxwQVuo6Qzp9CeLX-vKks5sijZ7gaJpZM4PYpyM
.

Well, this window was still in this state after about 3 hours (see last NEW REQUEST). So I assume no new calls have been made to selenoid. Is there any flag to have some more detailed logging?
headless-selenium-for-win: We tried headless-selenium-for-win but we couldnt watch the tests using vnc beacue of virtual desktops (at least we couldnt find an easy way).

@benzman81 could you check for process ID of running IE and send us all log entries containing this ID. Selenoid is logging when the process is launched and then terminated.

@vania-pooh currently all machines have been destroyed an recreated. So I have to wait for the issue again. I will to so when it occures next time.

Had this issue again. I also noticed that the test hangs in that case. Only continues if I close the browser manually.
I could get the PID out of the taskmanager but they did not occur in the selenium console out. Can you point me how to get the pid in the console out?

@benzman81 what I mean is that you should get the process ID from the task manager (if you have cygwin you can also type ps -W | grep iexplore) and then try to find all entries in selenoid log.

Yes, thats what I did, but no entries with this process ID in the selenium console out.

There should be at least an entry about started process.

@vania-pooh yes, I could see those, but they did not have a number anywhere that matched the process Id. The next run I get this issue, I will post the log anyway with the PID, maybe you can find some other useful info.

@benzman81 agree.

and here it is. a test just hung again. hope the files help

tasklist1.txt
selenoid1.txt

@benzman81 ok, will take a look tomorrow.

And here are the interesting log lines:
2017/09/15 15:34:19 [117] [PROCESS_STARTED] [2232] [556.6406ms]
2017/09/15 15:34:19 [117] [PROXYING_REQUESTS] [http://<IP_ADDRESS>:59454]
2017/09/15 15:34:19 [117] [SESSION_ATTEMPTED] [test] [http://<IP_ADDRESS>:59454] [1]
2017/09/15 15:34:26 [117] [SESSION_CREATED] [test] [a1be16b6-3f43-4d9d-a706-84da7641d8a1] [http://<IP_ADDRESS>:59454] [1] [6.3818359s]
2017/09/15 15:35:20 [SESSION_DELETED] [a1be16b6-3f43-4d9d-a706-84da7641d8a1]
...
2017/09/15 15:35:21 [117] [TERMINATING_PROCESS] [2232]
2017/09/15 15:35:21 [117] [TERMINATED_PROCESS] [2232]

But in your tasklist you can see that IEDriverServer.exe process 2232 still exists.
Image Name                     PID Session Name        Session#    Mem Usage
========================= ======== ================ =========== ============
IEDriverServer.exe            2232 Console                    1     14,024 K
iexplore.exe                  2012 Console                    1    264,592 K

So I think the issue is somewhere in process killing logic. We are logging when Golang fails to kill the process. So this even could be a Golang issue.
@aandryashin any more thoughts?

Just to maybe have more data I attach two more files. But looks the same.
selenoid2.txt
tasklist2.txt

In that case IEDriverServer process has ID 3504 with the following Selenoid log entries:
2017/09/16 07:56:47 [24] [LOCATING_SERVICE] [internet explorer-11]
2017/09/16 07:56:47 [24] [USING_DRIVER] [internet explorer-11]
2017/09/16 07:56:47 [24] [ALLOCATING_PORT]
2017/09/16 07:56:47 [24] [ALLOCATED_PORT] [51892]
2017/09/16 07:56:47 [24] [STARTING_PROCESS] [[C:\Users\IEUser\Downloads\IEDriverServer.exe --log-level=DEBUG --log-file=C:\Users\IEUser\Downloads\IEDriverServer.log --port=51892]]
2017/09/16 07:56:47 [24] [PROCESS_STARTED] [3504] [571.2891ms]
2017/09/16 07:56:47 [24] [PROXYING_REQUESTS] [http://<IP_ADDRESS>:51892]
2017/09/16 07:56:47 [24] [SESSION_ATTEMPTED] [test] [http://<IP_ADDRESS>:51892] [1]
2017/09/16 07:56:54 [24] [SESSION_CREATED] [test] [3f3c4f5e-3ea7-4a99-b541-3e67d648d0ea] [http://<IP_ADDRESS>:51892] [1] [7.3076172s]

I don't see any process termination log entries, but I think this is because the lines above are the last ones in the log, so they should be present. Aren't they?

Yes, they are the last lines in the log and the browser is open.
Maybe the issue is on our side in the tests. As I close the dialog that is open in our webapp, then the tests continues but fails. Is it somehow possible to log the json calls from selenium using selenoid? Maybe there are endlees calls going to the browser so that selenoid doesnt run into timeout?

Ok, now I found a while loop that keeps checking somthing in one test. After I change that, at least this was fixed. Anyway, is it somehow possible to log the json calls from selenium using selenoid? If so I would have found the issue better. Im using this configuration, but no driver log is written:
{
	"internet explorer": {
		"default": "11",
		"versions": {
			"11": {
				"image": [
					"C:\\Users\\IEUser\\Downloads\\IEDriverServer.exe", "--log-level=DEBUG --log-file=C:\\Users\\IEUser\\Downloads\\IEDriverServer.log"
				]
			}
		}
	}
}

We will check logging logic on Windows and keep you updated.
2017-09-16 12:14 GMT+03:00 Markus Krüger<EMAIL_ADDRESS>
Ok, now I found a while loop that keeps checking somthing in one test.
After I change that, at least this was fixed. Anyway, is it somehow
possible to log the json calls from selenium using selenoid? If so I would
have found the issue better. Im using this configuration, but no driver log
is written:
{
"internet explorer": {
"default": "11",
"versions": {
"11": {
"image": [
"C:\Users\IEUser\Downloads\IEDriverServer.exe", "--log-level=DEBUG --log-file=C:\Users\IEUser\Downloads\IEDriverServer.log"
]
}
}
}
}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aerokube/selenoid/issues/216#issuecomment-329956764,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABWNbZ9eiDKbP_IogAHzXMoV_IKDijS0ks5si5FagaJpZM4PYpyM
.

@benzman81 for logging there is https://github.com/aerokube/selenoid/issues/68.

So, for now there are no more tests hanging, but I can still see that browsers stay open on some vms. Seems like they have been trying to be initialized. I attach the files of two machines, maybe it helps.

Selenoid1.txt
Selenoid2.txt
TaskList1.txt
TaskList2.txt

On Windows you can tune two interesting settings:

Turn off TCP offloading (https://support.rackspace.com/how-to/disabling-tcp-offloading-in-windows-server-2012/)
Max TCP connections per host in IE (https://support.microsoft.com/ru-ru/help/183110/wininet-limits-connections-per-server)

Not sure whether it is related to your issue but according to our experience this can dramatically accelerate things.

Well, that didn't help, but didn't make it worse ;-)
Today we had a similar issue, but with an existing browser AND IEDriverServer.exe running. The big problem here was, that our tests hung again. A browser kill didn't help, but after killing the IEDriverServer.exe they continued running. Irritating is, that the process of the running IEDriverServer.exe never occurs in the selenium log :-/ (see attachments). Maybe a release with passthrough of logs from IEDriverServer.exe would be helpful to investigate such issues (including documentation on how to do it).
Selenoid3.txt
TaskList3.txt

@benzman81 ok, will try to release Selenoid as soon as possible.

@benzman81 released 1.3.7 with driver logs capturing support.

@benzman81 sorry, forgot to ask - did detailed IEDriver logs help to find anything new regarding this issue?

@vania-pooh yes, it helped a lot. I could find some issues in my code. If I have any further problems or notice the open windows again, I get back here and reopen.


SVG Size

I've been playing around with Heroicons UI and Tailwind today, but I'm running into a weird issue with the SVG size. When I insert the SVG inline and assign some Tailwind classes, the icon's path is half the size of its parent svg element. I can bump up the sizing on the svg element, but that leads to some really strange line height and padding issues.
When I put a Zondicons icon in with the same Tailwind classes, both the svg and path elements have the same size like I would expect.
https://jsfiddle.net/9zgf0yLf/1/
Am I doing something wrong or is there some distinct difference between Heroicons UI and Zondicons that I'm not seeing. Thanks for any help!

You are not alone

It seems that if I manually set the viewBox to 2 2 16 16 on something like x-circle then it respects the sizing.
It drives me crazy that if I'm using tailwind to size the heroicon I'll end up wanting something that's 16x16 (w-4) so I have to make the outer container 20x20 (w-5) so that I end up with a 16x16 icon.

Opened: https://github.com/tailwindlabs/heroicons/issues/664


add logging context manager and decorator

the PR adds support to contextual logging, the trace context-manager or decorator can be used.
It handles both threaded and async code.
Example usage:
import unify
unify.activate("some project")

def inner_fn(data, st):
    time.sleep(st)
    unify.add_log_entries(d2=data)

@unify.trace()
def fn1(data1, data2, st):
    unify.add_log_entries(d1=data1)
    inner_fn(data2, st)

import threading

# Create two threads

thread1 = threading.Thread(target=fn1, args=("Thread-1", "data1", 1))
thread2 = threading.Thread(target=fn1, args=("Thread-2", "data2", 2))

# Start the threads
thread1.start()
thread2.start()

# Wait for both threads to complete
thread1.join()
thread2.join()

This will log two logs, with their respective data.
you can also use it as a context manager:

with trace():
    # any call to unify.add_log_entries will be logged to the same log

behind the scenes, the context manager/decorator makes a unify.log call and holds the returned (empty) log in a context-aware variable that was initially set as None, each subsequent call to unify.add_log_entries will refer to that log if no explicit id is passed, once the function finishes execution (or the context manager exists in the non-deocrator case) the variable becomes None once again.

Worth installing pre-commit and re-running with black enabled, but otherwise looks great 👌


Updates apollo-link dependency

This is necessary to prevent errors when using rollup to bundle

Not sure why travis is failing. I was able to pass tests following npm i

Right.. you're using yarn. NP
Also clobbers TS to make up for the Observer type going AWOL.

Codecov Report

Merging #4 into master will increase coverage by 0.2%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master       #4     +/-   ##
=========================================
+ Coverage   97.01%   97.22%   +0.2%
=========================================
  Files           1        1
  Lines          67       72      +5
  Branches       16       16
=========================================
+ Hits           65       70      +5
  Misses          2        2

Impacted Files
Coverage Δ

src/DebounceLink.ts
97.22% <100%> (+0.2%)
:arrow_up:

Continue to review full report at Codecov.

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

Very cool, thanks @hwillson 🎩

No problem @bennypowers - Observer is no longer directly exported from apollo-link. It can be pulled in from zen-observable-ts. Thanks!

Just wondering is there any plans to get this merged? We are using apollo link 1.2.3 so would be great to have debounce too.

Thanks a lot, and sorry for the delay guys, merging this now!


Add is_conformal method to Basis and Transform2D

This PR adds a new method is_conformal to Basis and Transform2D.
This method returns true if the basis is conformal, meaning it preserves angles and distance ratios, and may only composed of rotation or uniform scale. Returns false if the basis has non-uniform scale or shear/skew.
This can be used to validate if the basis is non-distorted, which is important for physics and other use cases. I have wanted a method like this for a long time but I did not know what to call it before now.
This PR also includes documentation and test cases. Most of the added lines are test cases.
I plan to use this method in the engine in #79364, but I wanted to open this PR to add the method first.

Updated the PR to have an optimized 3D implementation suggested by @SlugFiller.

Thanks!


Student Explorer sends email alert when advisor accesses bookmark for prior term courses

Student Explorer only displays course for current term. When advisor attempting to access a prior term course, maybe through a bookmark, system generated the following error message and sent out alert email:
Internal Server Error: /students/<advisor_id>/class_sites/<class_id>/
DoesNotExist at /students/<advisor_id>/class_sites/<class_id>/
StudentClassSiteStatus matching query does not exist.
Request Method: GET
Request URL: http://studentexplorer.tl.it.umich.edu/students/<advisor_id>/class_sites/<class_id>/
...
The desired fix would be catch the attempt as log warning, instead of sending out email alert.

I would prefer to continue receiving email alerts unless we change the template to better explain to them what is going on

Hi @zqian @mfldavidson , I am currently enrolled in EECS 481 at umich, and for our project my partner and I have to make an open-source github contribution. Would it be ok if we worked on this issue / is it still open? Thanks!

Hi @mdhvprasad I think we need to think some more about what we want to do about the issue @zqian raised--can you take a look at some other issues?


Implement an egress-policy: warn-only mode

https://github.com/microsoft/msquic/pull/2310#event-5927310870
It should not block traffic, but should warn if there are new endpoints.

I would like to request this "warn-only" mode as well, in addition to the existing feature of traffic blocking.  The use case I ran into is that our github actions workflows contain outbound endpoints referring to AWS region, such as "lambda.us-east-1.amazonaws.com:443".  But we don't use the same region for our different AWS accounts, resulting additional outbound endpoints need to be added to the list so traffic doesn't get blocked when running for accounts pointing to a different region than the one defined in the endpoint urls.
I think it's a good idea to let the users/companies decide which approach they are comfortable with.  For companies who only deal with a set number of limited regions, adding the respective outbound endpoints for each region is not unreasonable.  The problem is that it can be easily missed.  Users may not be watching closely enough the region value in the endpoints, ended up traffic getting blocked unintentionally.  Warn mode is definitely nice to have, especially for companies who deal with many different regions for different accounts.


Fix URL parsing in Firefox

Closes #3193.
In Firefox 77, inheriting from URL seems to cause the errors. This PR works around it by having an internal URL object that most fields are proxied to ("Facade pattern") ("Adapter pattern").
This does indeed fix both errors from #3193: assign works again, and there is no console error anymore.

Too verbose. If extend isn't possible, I'd rather just have a read-only url property rather than all that.
And really I don't understand why the current solution works... sometimes. Either it works or it doesn't. Is it a browser bug? Should it be reported?

Could this be the problem?
https://github.com/sindresorhus/refined-github/pull/3153/commits/c01ed3d29773c3f5d1b08004b0c83549393fe1d2
or the following
https://github.com/sindresorhus/refined-github/pull/3153/commits/effebd55f026752ca6d254577a7cbadd79559734

Until someone finally figures this out, perhaps your original solution was cleaner. I dropped some unused method


bug: api.NewClient account format is confusing

Describe the bug
When I tried to use this, I used the account string straight from the json file i downloaded when creating my api token.
That account string is in the format of company.lacework.net, which led to me failing to create the client instance since the url it used was https://company.lacework.net.lacework.net.
With the code as it is, if someone wanted to use an EU tenant, they'd have to provide the account in an even more unexpected format company.fr for it to work.
To Reproduce
Steps to reproduce the behavior:

Get account string as it's specified in the downloaded api token's json file.
try to create a new client with account from above, and see it fail.

Expected behavior
This line should be changed to check if the account already ends with lacework.net and not add it if it's already present.
Screenshots
N/A
Please complete the following information):

OS: [e.g. macOS Big Sur 11.5.2 ]
Version v0.38.0

Additional context
Add any other context about the problem here.

@hal-y thank you so much for your feedback! We will get this fixed in our next release.