[New game]: Your favorite photo's jigsaw puzzle

🎮 Game Request
This game would work like a simple jigsaw puzzle game.
But, the catch is, that it would allow the user to choose their favorite photos from their respective galleries and convert it into a jigsaw puzzle. This would turn out to be a really interesting game.
Point down the features
Game points

The user gets to choose a picture from their gallery and convert it into a puzzle and then solve it.

The user gets to select the number of pieces, according to his capability.

The feature of cool songs being played in the background will also be enabled.(A continuous loop of pre-recorded songs, written in the code).

A cool message displayed on winning the game, and overall a fun experience.

Select program in which you are contributing
No response
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @daydreamer15 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @daydreamer15 👋!
@kunjgit here,
We have this annoucement to make 📢!!
During the time of PR don't alter main index file ! 🙂
In main Readme file just use * as a game number during the time we are merging we will add number !! 👀
Make sure you join our Discord🕹 if you haven't 😇
Thank you 💗!

Hello @daydreamer15, Time's Uppp!⏰
Sorry for closing your issue!
But it's more than a week since we haven't received anything from your side 😢 .
Come up with new ideas, create a new issue and make sure you finish it within a week! 🔥
All the best! 🚀
Happy Hacking! 💗

Hey @daydreamer15 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Wrong org name in dependencies?

I am trying to use KEthereum, but my build is failing because gradle is unable to find Kbignumbers. From the error I think I managed to trace the issue, and I believe the first kbignumbers here and here should say komputing instead. Is that the case?
EDIT: probably not the case.
The gradle ERROR:
Execution failed for task ':jvmTest'.
> Could not resolve all files for configuration ':jvmTestRuntimeClasspath'.
   > Could not find org.kbignumbers:kbignumbers-jvm:1.0.0-RC1.
     Searched in the following locations:
       - https://repo.maven.apache.org/maven2/org/kbignumbers/kbignumbers-jvm/1.0.0-RC1/kbignumbers-jvm-1.0.0-RC1.pom
       - https://repo.maven.apache.org/maven2/org/kbignumbers/kbignumbers-jvm/1.0.0-RC1/kbignumbers-jvm-1.0.0-RC1.jar
       - https://jitpack.io/org/kbignumbers/kbignumbers-jvm/1.0.0-RC1/kbignumbers-jvm-1.0.0-RC1.pom
       - https://jitpack.io/org/kbignumbers/kbignumbers-jvm/1.0.0-RC1/kbignumbers-jvm-1.0.0-RC1.jar
       - https://dl.google.com/dl/android/maven2/org/kbignumbers/kbignumbers-jvm/1.0.0-RC1/kbignumbers-jvm-1.0.0-RC1.pom
       - https://dl.google.com/dl/android/maven2/org/kbignumbers/kbignumbers-jvm/1.0.0-RC1/kbignumbers-jvm-1.0.0-RC1.jar
     Required by:
         project : > com.github.komputing:kethereum:0.76.2 > com.github.komputing.kethereum:keccak_shortcut:0.76.2 > com.github.komputing.khash:keccak-jvm:1.0.0-RC2

Just saw this is a duplicate of https://github.com/komputing/KEthereum/issues/71. Closing.


Add type hints support

Don't know why but my black does not show any warning

Thanks for your contribution. Let us ignore the style issues for now.


Adding armc6 support

adding armc6 support so we can use utensor in mbed studio

@mbartling - over here now
I dont think its worth adding ARMC6 to the CI as there is no way for you to run ARMC6 in CI (requires so much excessive BS to get license keys enabled, same reason you dont have support for ARM or IAR in there)


added supp for serbian language

is pretty much the same language as bosnian and croatian. changed a couple of lines :)
Hello there!
Make sure you've followed the Contributing guidelines before finalizing your pull request.
TL;DR:

[x] You're opening this PR against the current release branch
[x] Works on both Python 2.7 & Python 3.x
[ ] Commits have been squashed and includes the relevant issue number
[ ] Pull request description contains link to relevant issue or detailed notes on changes
[x] This must include example code demonstrating your feature or the bug being fixed
[x] All existing tests pass
[x] Your bug fix / feature has associated test coverage
[ ] README.md is updated (if relevant)

Thanks!


Beta documentation for Snowflake Iceberg tables

What are you changing in this pull request and why?
This PR accomplishes a few things:

Adds the Iceberg table format beta to the Snowflake configuration reference
Moves all of the table definitions to the top of the doc so they're closer together.

Checklist

[ ] I have reviewed the Content style guide so my content adheres to these guidelines.
[ ] The topic I'm writing about is for specific dbt version(s) and I have versioned it according to the version a whole page and/or version a block of content guidelines.
[ ] I have added checklist item(s) to this list for anything anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."

Approval from @amychen1776 to move forward with merge


Update dependencies to fix bitcode linker errors when building for device

This change updates both Mixpanel and HockeySDK to new minor versions that include support for bitcode.

This build failure was hidden because CI doesn't currently build for device. I'm going to look into setting up a device build as part of this PR.

👍 Should be possible to build for device by using platform=iOS/Generic in the build script. Maybe have it build and run the tests, then do a second build for device?

I've pushed a new bin/archive script that ultimately performs an xcodebuild archive. The archive action builds for device, and I think having a successful archive step is a stronger signal from CI than a build that targets the generic device destination. Let me know what you think.

makes sense to me 👍


Disable dates not possible

Use the latest version and can not disable dates. Even on the documentation page http://amsul.ca/pickadate.js/date/#disable-dates the dates from the sample code are not disabled.
Only weekdays are possible to disable.

I just came here looking to see if anyone else had an issue and then started to investigate in the source and realised I was setting the option 'disabled' not 'disable' by mistake, so ensure you aren't doing the same.
I can vary the disable option is working in the latest version, looking at the documentation that you mentioned it is also working there it just appears not to be because the dates are set in 2015 and the picker automatically shows the current (2016) date when opened. If you click back to 2015 you'll see it working so unfortunately if it's not working on your implementation it is something about the way you are doing it.
I believe this should be closed.

Thanks
I realize that the month count did irritate me. Yes, it works when I consider this count. But thing is, the disabled dates come from a db with dates in usual count where march is 3. Now we have to manipulate the month after reading the db with our dates. Why is the mointh count that strange?
And why can it not accept dates in the format we have defined? Like in our case it is:  format: 'dd.mm.yyyy'. Other datepickers do so.
For our project I had to take something else. But your datepicker would still be the one we like the most..

Ye I'm not sure why it takes only those date formats, although to be honest when you have to setup custom JS to use it anyway (the settings object is pretty unique to this datepicker) it isn't a lot of effort to do (date.month-1).
However I'm sure the author would love for you to implement the feature of other dates and make a pull request. Remember this is open source so if something doesn't fit you can always make the adjustment and more than likely other people will be grateful for it. (Although in this case the documentation could really do with updating to use dynamic dates so it doesn't appear broken).
Also having look at the documentation it does say this:

NOTE: months in a JavaScript Date object are zero-indexed. Meaning, new Date(2015, 3, 20) is 20 April, 2016.

To stay consistent with this, whenever an integer is used in reference to a month, pickadate treats it as zero-indexed. Dates as strings are still parsed as expected.

I understand what you mean. But I am not that experienced in JS to really do adjustments. I can close this for now.

You absolutely must put this in the documentation.
I imagine the cumulative lost time on "head scratching", from the many people who've struggled with this same issue, must be massive.


Inat Import Stimulus

Uses Stimulus controller and Turbo to update the JobTracker display.

coverage: 93.472% (-0.04%) from 93.511%
when pulling ac563accb5f618d545db6d204366fb9175cf3a38 on jdc-2319-stimulus-inat-import
into bd075db495bdb78e909fece91be88ecc428ae6d1 on main.


manage.py error

Traceback (most recent call last):
File "manage.py", line 10, in
execute_from_command_line(sys.argv)
File "/usr/local/lib/python2.7/dist-packages/django/core/management/init.py", line 453, in execute_from_command_line
utility.execute()
File "/usr/local/lib/python2.7/dist-packages/django/core/management/init.py", line 376, in execute
sys.stdout.write(self.main_help_text() + '\n')
File "/usr/local/lib/python2.7/dist-packages/django/core/management/init.py", line 242, in main_help_text
for name, app in six.iteritems(get_commands()):
File "/usr/local/lib/python2.7/dist-packages/django/core/management/init.py", line 109, in get_commands
apps = settings.INSTALLED_APPS
File "/usr/local/lib/python2.7/dist-packages/django/conf/init.py", line 53, in getattr
self._setup(name)
File "/usr/local/lib/python2.7/dist-packages/django/conf/init.py", line 48, in _setup
self._wrapped = Settings(settings_module)
File "/usr/local/lib/python2.7/dist-packages/django/conf/init.py", line 132, in init
mod = importlib.import_module(self.SETTINGS_MODULE)
File "/usr/local/lib/python2.7/dist-packages/django/utils/importlib.py", line 35, in import_module
import(name)
File "/root/NewsBlur/settings.py", line 567, in
RAVEN_CLIENT = raven.Client(SENTRY_DSN)
File "/usr/local/lib/python2.7/dist-packages/raven/base.py", line 147, in init
dsn_config = raven.load(dsn, transport_registry=self._registry)
File "/usr/local/lib/python2.7/dist-packages/raven/conf/init.py", line 53, in load
scope_extras = transport_registry.compute_scope(url, scope)
File "/usr/local/lib/python2.7/dist-packages/raven/transport/registry.py", line 57, in compute_scope
return transport.compute_scope(url, scope)
File "/usr/local/lib/python2.7/dist-packages/raven/transport/base.py", line 188, in compute_scope
raise ValueError('Invalid Sentry DSN: %r' % url.geturl())
ValueError: Invalid Sentry DSN<EMAIL_ADDRESS>

Like the error implies, you'll want your own Sentry DSN. Or just turn off Sentry, which is used as the error logger. I disable Sentry in DEBUG.

https://github.com/samuelclay/NewsBlur/blob/97b0910e675c79d376e7988234bebc7f692b687d/settings.py#L560
It's enabled when is NOT development mode. How can I disable it?

Why not run with DEVELOPMENT = True?


Could you make switch ctrl and alt key function for signal, or vim shortcut key?

Description of the new feature/enhancement
Please, Could you make change ctrl to alt key function for signal, or vim shortcut key?
Proposed technical implementation details (optional)
Windows is almost use ctrl key Such as Ctrl+1, Ctrl+2, ctrl+c, ctrl+v, ctrl+l, ctrl+r.
This makes my little finger hurt for keep press ctrl key.
So, I used replace ctrl and alt keys on Windows.
Could you make switch ctrl and alt key function for signal, or vim shortcut key?
In Mac, iterm2 terminal is almost use cmd key for window function.
But, In order to signal, or vim shortcut key almost use control key.
This is very efficient.
If you make this function, Many people possible for string copy with ctrl + c key.

Thanks for posting! I'm not sure I understand the request. If you are asking us to allow you to change Ctrl to Alt inside Terminal, you can already do that today by changing the key bindings. They are in the JSON file and in the settings UI.

Yeah, I knew that. But, one key bindings impossible.
Or, If I'm wrong my configuration, Please tell me.
{ "command": { "action": "ctrl"}, "keys": "alt"},
{ "command": { "action": "alt"}, "keys": "ctl"},

Yeah, I think, You don't understand.
How about this?
{"command":{"action":"sendInput","input":"\u0003"},"keys":"alt+c"},
{"command":"copy","keys":"ctrl+c"},
If you use this json, You can string copy with ctrl + c.
If you want to interrupt, You can use interrupt with alt + c.
This means, ctrl and alt keys switched.
But, Actually, I want to only switch ctrl and alt keys.

Hi @CatCatCatDog. Maybe PowerToys may be of interest to you! It has a keyboard manager that lets you do exactly this (and much more). Here's a link to the relevant docs: https://learn.microsoft.com/en-us/windows/powertoys/keyboard-manager

Are you looking for a setting to change the way the Terminal encodes ctrl/alt, so that pressing ctrl+key would instead send alt+key to the connected commandline application? (e.g. you'd need to press alt+z to send a ^Z to the client application)?

Being able to remap modifier keys is exactly the feature I would love to see. For example, I would like to swap ctrl and alt in Windows (via PowerToys, etc) but swap them back in Terminal for Linux use.


[BUG] Harcoded indexes present in 3 macros

This is a tiny bug - but skews some healthcheck type searches I'm writing & may cause false negatives for customers.
Present in ESCU version 4.44.0
These 3 ESCU sourcetype macros all have hardcoded indexes.

f5_bigip_rogue
zeek_rpc
zeek_ssl

Thanks @dluxtron ! - fixed these in here https://github.com/splunk/security_content/pull/3247


[Bug] perset.yml is not created when reading is not finished

While I am running npm run e2e I am getting a few Errors like this:
2020-12-02T11:10:49.407Z error [AUDIT_LOG]  Transaction 9586573 type 3 ignored. Multisign for destination could not be created: 9489C6500225CECB1CF3A4264D3CF93620DD5D64CDF7627441D7376B7B32E546. Payload:<PHONE_NUMBER>000000359454C7756578BC75BE772BB8D22D32A2D8394AECCFEF19AFFC8136FC7D124B50C3723B391486EE93D7ECE6899E4EA73288B43436FBAA41C9A7C4ACE6CF56019489C6500225CECB1CF3A4264D3CF93620DD5D64CDF7627441D7376B7B32E546000000000168414100000000000000000100000000000000960F1420508807D3470E9B0741052C28F4AD481A469A1AEDF166330A7A86D061800000000000000080000000000000009489C6500225CECB1CF3A4264D3CF93620DD5D64CDF7627441D7376B7B32E5460000000001685541020203000000000068B95C6B8D48E225BD9A2D13C5503AB16FFFA73551FF50DC686CB231A6763C54FF3877AB0768CFEFD7548287C5DBF6C26867F5D8FCAF5C41EF0C0EE7A71C1CACF808DCECE7975060. Error: RangeError: Maximum call stack size exceeded

@NikolaiB I think this is a catapult-opt-in bug, not a bootstrap bug.  The collection tool didn't finish correctly so bootstrap complains there was no preset.


Untracked flag usages in daa instruction

(this issue is very similar to https://github.com/santiontanon/mdlz80optimizer/issues/16)
The instruction daa requires the flags to be properly set/reset to work (just the half-carry flag?). This set-up is usually performed by a add/adc/dec/inc/neg/sub/sbc instruction; so if the number is "just read", a fake operation (such as add a, 0) must be inserted. Therefore, the suggestion to "Remove unnecessary add a,#00 (2 bytes, 8 t-states saved)" is not valid:
Example code from PyramidWarp.enhancedplus.asm:
; Prints the room number
	ld	a,(game.current_room)
	add	a,00h
	daa

This example code does not behave as expected without the add a, 0.

Oh, thanks for the report! I just checked, and similar issue as with issue #16 , the instruction set file is missing the flags that daa depends on! I'll fix it later tonight as well. I added all the instructions dependencies by hand going through the z80 manual. So, I'm sure there was some "human error" there, and I missed some. Hopefully those are corrected bit by bit, and eventually the file is 100% correct :)

Alright, resolved this too! :)


Issue logging seaborn / plt plot via wandb.log()

Describe the bug
When I attempt to log a seaborn scatterplot to weights and biases I get the following error:
*** AttributeError: 'XAxis' object has no attribute '_gridOnMajor'

To Reproduce
tips = sns.load_dataset("tips")
fig, ax = plt.subplots(figsize=(11.7, 8.27))
sns.scatterplot(data=tips, x="total_bill", y="tip")
wandb.log({"test_plot": fig})
# or
wandb.log({"test_plot": plt})

Expected behavior
Logging the plot to wandb interface in plotly format
Operating System

OS: MacOS
Browser: Chrome

Additional context
This seems to be happening with the example given here too.

Hey @harritaylor
I tried reproducing the error, but everything checks out fine for me.
Code
import seaborn as sns
import matplotlib.pyplot as plt
import wandb

with wandb.init(entity="xxx", project="xxx") as run:
  tips = sns.load_dataset("tips")
  fig, ax = plt.subplots(figsize=(11.7, 8.27))
  sns.scatterplot(data=tips, x="total_bill", y="tip")
  wandb.log({"plot":fig})

Screenshot

Hope this helps.

Hey @harritaylor
We are closing this issue as of now.
Please feel free to open this if the issue persists. :smile:

@ariG23498 thanks for getting back to me, apologies I didn't reply sooner - I only got back to work today.
I think it was something to do with my environment -- with a fresh conda env it works fine. Sorry!

I had an issue like this because my matplotlib and plotly version is incorrect:
In particular I hit this bug when using the following versions of matplotlib/plotly versions.
matplotlib>=3.3.3
plotly>=4.12.0

@harritaylor Do you mind sharing your matplotlib and plotly versions? Thank you!

Hi @harritaylor @chanmi168 I got the same issue, can you share your matplotlib and plotly versions?  Thanks!

@chanmi168 FYI I fixed this issue with following versions:
matplotlib == '3.3.3'
plotly == '5.1.0'
Hope it helps.

@abcde-1447
Thanks so much!


Module not found: Error: Can't resolve '@angular/elements'

I'm getting this error when we include
import { ReteModule } from 'rete-angular-render-plugin';
this line in our module.
ERROR in ./node_modules/rete-angular-render-plugin/dist/esm5/module.js
Module not found: Error: Can't resolve '@angular/elements' in '/Volumes/Data/admin-ui/node_modules/rete-angular-render-plugin/dist/esm5'
may be this package is not getting added in dist while building this application.

you'll need to install this one manually if you have not already.
npm i --save @angular/elements


Need direct messages.

Is there any chance to get one?

@quantumKernel
Your issue report does not conform to the issue template that has been specified for this repo:
https://raw.githubusercontent.com/ping/instagram_private_api/master/.github/ISSUE_TEMPLATE.md
Please edit your issue to comply with the template requirement.
This issue will be closed after 24 hours if no followup action is taken.

[This comment is auto-generated. ref=notemplate]


Errai SDM - Code refresh doesnt compile HTML Files

Hi Errai Team,
Errai Version: 4.0.0-Beta3
GWT Version: 2.8.0
When running Super Dev Mode, and I changed something inside .html file and refresh the browser. It seems that Errai does not update my code changes. If I changed both .html and .java for example it's working back.
Best Regards,
Kevin

Hi @kevzlou7979,
I've just verified that this works with our tutorial, so I am closing this issue.
I'd suggest comparing your configuration to our tutorial, or asking for help on our forum.
If you would like to file a bug report, please do so through JIRA. That being said we will need more details (ideally a reproducer) to be able to act on it.
Cheers.


Strange Graphical Error

debug log
After clicking on anything in the in-game mod menu (the one accessible from the Mods tab on the menu) at all, almost all text in both the mod menu as well as the game itself become scratchy and low-resolution. Sometimes it resolves itself seemingly at random, other times it takes closing the game entirely to fix it. Regardless, this makes navigating the mod menu somewhat irritating to the eyes.

are you using optifine?

i have no idea. i've noticed it when testing forgery ages ago. but Una policy was no touch optifine issues (persumably because their all a nightmare to deal with). and considering i'm much worse at render code i fully support it.
sorry, you could try optifine dev i guess?

It's all fine, no need to apologize. I'm using the latest Optifine version for MC 1.20.1, so presumably there's not much I can do. If it's ok to ask here, do you know any alternatives that'd let me use Complementary Shaders?

looks like iris/sodium ports for forge exist and are called oculus and rubidium. so try those i guess? not up to date on the current mc stuff though, just figured out 1.20.5 exists looking for an iris port.


What's the meaning of β in Appendix B?

In Appendix B, Eq. (17), the paper writes that:

Later the paper states that "We set both β_1 and β_2 to 0.8 ."
I was wondering why we can assume P(a=a_R|F) = (1-β_1), and why β_1 = 0.8?

Hi @coderlemon17
The motivation behind using Bayes' Therom (Appendix B) instead of simple counting (i.e. non_factual_score = count_not_match / (count_match + count_not_match)) is because in our initial experiments when we applied unanswerability scores to filter out bad questions, some instances ended up with a smaller number of questions.
For example, [case1]: count_not_match = 2, count_match = 1 and [case2]: count_not_match = 12, count_match = 6, both give the same nonfact_score (e.g., 2/3) if we do simple counting. In contrast, if we use Bayes', it will take into account the amount of evidence, i.e. count_not_match = 12, count_match = 6 will yield a higher non-factual score than count_not_match = 2, count_match = 1
beta_1 is the probability that the two answers (a' and a_R) are not the same given that the sentence (to be evaluated) is False / non-factual. The value of beta_1 would be 1.0 if we assume that both QG and QA are perfect, while the value will be smaller if we assume that the systems can make errors. In other words, if we set the value to be high (e.g., close to 1.0), the effect of the amount of evidence will be higher. We tried 0.7, 0.8, 0.9 in our initial investigation, and it seems 0.8 was a reasonable value (although 0.7 and 0.9 would work too).
I hope this helps clarify the paper
Potsawee

Hi @potsawee, thanks for your reply.
Yes, I understand why you use Bayes' Theorem here, but my question is "why can we assign an arbitrary value to β_1/2". Like you said, β_1=P(a≠a_R|F) is probability that QAs will make different choices given hallucinated contents.
So my doubt is, since QGs and QAs are fixed models, probability P(a≠a_R|F) should be a fixed (but unknown) value instead of one that can be arbitrarily chosen, right? For example, if we have very poor QGs and QAs, then β_1 should be low (instead of chosen 0.8 in the paper).

Hi @coderlemon17 ,
Thank you very much for your comment, and I agree with your point that the current setup of selecting beta values is not great.
and yes, QGs and QAs are fixed, and Prob(a != a_R | F) can be computed empirically. But since we only have one evaluation dataset, tuning the beta values on this evaluation dataset would be somewhat cheating. We believe to do this properly we would have to collect some more data points to tune these beta values. In this work, we only tried a few beta values at the beginning and we stick with 0.8.
Best wishes
Potsawee

Hi @potsawee, thanks for your clarification😄, closing issue now.


nvim-tree current directory text is too dark to see

The cwd text in nvim-tree doesn't show up anymore. The text color is too dark. This first started happening a few weeks back with an update of catppuccin.

The text is still there, but it's normally too dark to see.

This doesn't happen with other color schemes--only catppuccin.

I've had the same issue — it seems to update to the correct color upon executing :Catppuccin {flavor}.

@drinkitup777 that's intentional. You can enable that by setting the following in the intergrations section of your Catppuccin conf:
nvimtree = {
	enabled = true,
	show_root = true,
}


perf: improve HShots.to_pytket performance

just a drive by I happened to do while debugging something else. Performance gains are not a bottleneck but it is more readable so might as well.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.48%. Comparing base (f4e5655) to head (c99d6c0).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #647   +/-   ##
=======================================
  Coverage   92.48%   92.48%
=======================================
  Files          66       66
  Lines        7492     7492
=======================================
  Hits         6929     6929
  Misses        563      563

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


[Feature Request] Add downloadTo to Storage Client

Feature Request
The Java Cloud Storage client google-cloud-storage has a sample to download a Blob (shown below). The feature request is to add this functionality into the sample shown into an additional method downloadTo for Class Blob to reduce complexity of downloading a blob using the Java Storage client. At the moment other clients use this approach and allow a user to download a given Blob to a specified file.

Language Comparison:
In other client libraries (google-cloud-ruby) there exists a method to download a Blob to a specified file.

Another example using .NET:

Example of usage
try {
  String bucketName = ”BUCKET_NAME”;
  String blobName = ”BLOB_NAME”;

  Blob blob = BlobId.of(bucketName, blobName);
  Path path = FileSystems.getDefault().getPath("output.ext”);

  blob.downloadTo(path);
} catch (IOException io) {
   //...
}

@garrettjonesgoogle is this being prioritized this quarter?

@frankyn i'm not sure if you are asking whether we will decide on the prioritization this quarter (maybe) or whether we are scheduling it for this quarter (not yet).

I mean, is it scheduled for this quarter. This issue will help the developer experience when using the Java Client library to download a blob. Thanks @garrettjonesgoogle


Update MD file on new branch 2017-12-10_15-31-22

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Remove unimplemented interface CacheProvider and all references to it

Fixes #3289
Description
Removes the CacheProvider interface from our dspace-services module, along with all references to it.  This interface is unimplemented in our codebase (and didn't exist in v6 or v5 either).  As the ticket notes, we now receive a detailed stacktrace WARN in our logs each time DSpace starts up, as it cannot find a CacheProvider bean.   So, this fixes that issue by removing the CacheProvider altogether.
Instructions for Reviewers

Review the code changes
Build and start up DSpace, verifying the detailed warning no longer appears (it would appear during Ant build as well as when starting Tomcat).

In the future we may want to scrap the whole CachingService in favor of JSR 107 (JCACHE).  Recent Hibernate releases prefer JCACHE to their product-specific adapters, and recent EHcache implements it.

@mwoodiupui : Agreed, I was just trying to implement a "quick fix" in this PR. But, yes, I think we may want to investigate the usefulness of the "dspace-services" module in general.  There are a few services there that might still have usefulness, but others might be unnecessary abstractions... so, I'd welcome more cleanup work in this area.
By the way, if you (or anyone else reading this) has a chance to give this a quick review, I'd appreciate it.  It "looks safe" to me, but I definitely want a second set of eyes before we merge this.

Merging with 1 approval. Thanks @mwoodiupui for testing this out as well!


Fix syntax error

On PHP 5.5.38 I get a syntax error unexpected '[' here. Not sure exactly why since array dereferencing was added in PHP 5.4. But this fixes it.

Could I ask for one small tweak? I'd like for $container to be $app since that is what we usually call it in docs and so on, so that makes it easier to see what it is if one does not understand what getContainer actually gets. If you don't want to do it just lemme know I'll do it after merge :)

OK done. I couldn't see how to squash the commits into one from the GitHub interface, but I think you can do that your side?

Nah, we'll live with the commits ;)


Web - Login page - Stack with 'E' logo under specific Mac/Safari versions

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.63-7
Reproducible in staging?: y
Reproducible in production?: n
Issue reported by: Applause - Internal Team
Action Performed:

Open https://staging.new.expensify.com/
Try to login

Expected Result:
The user should be able to login without any issues
Actual Result:
Stuck with 'E' logo under Specific Safari versions:
Safari 16.1 Released October 24, 2022 / macOS 13.0
Safari 15.6.1 Released August 17, 2022 / macOS 12.5
Workaround:
n/a
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/347721d6-460e-487a-8060-4e8b0b2c3cf6
View all open jobs on GitHub

I tested in staging and I can open the App just fine. If the issue only happens in those specific version, I'm just gonna close this issue since those are really outdated Safari versions from over 2y ago.


Error during new Release Check

I am on Version<IP_ADDRESS>809092115 and when I launch the Dbeaver software on my machine it tries to check for updates and during that it throws a JAVA Null Pointer exception
see attached snapshot for details

I was facing this issue as well. Manually updating to release 5.2.1 solved the problem

I'm getting it as well.


autodiff-based deconvolution

Depends on https://github.com/NervanaSystems/ngraph/pull/1173

TESTNOW

TESTNOW

TESTNOW


vuejs site running in development mode

Reproduction link
https://vuejs.org
Steps to reproduce
enter https://vuejs.org in browser address bar, check console in browser development tools
What is expected?
vuejs site running in production mode
What is actually happening?
vuejs is running in development mode

Or is it supposed to run in development mode?

It looks like Evan beat me to this. 🙂 Should be fixed now.

Isn't it a "feature", as of #1140?

Yes, we just turned off the warning so people don't have to worry about it.


Annotate count

Description

Motivation and Context

How Has This Been Tested?

Prior art for annotating plans:
https://github.com/bluesky/bluesky-queueserver/blob/main/bluesky_queueserver/manager/profile_ops.py#L2468```python

from bluesky_queueserver.manager.profile_ops import _process_plan as ser_plan
import bluesky.plans
ser_plan(bluesky.plans.count, existing_devices = {}, existing_plans = {})
{
'name': 'count',
'properties': {'is_generator': True},
'parameters': [
{'name': 'detectors', 'kind': {'name': 'POSITIONAL_OR_KEYWORD', 'value': 1}, 'description': "list of 'readable' objects"},
{'name': 'num', 'kind': {'name': 'POSITIONAL_OR_KEYWORD', 'value': 1}, 'description': 'number of readings to take; default is 1\n\nIf None, capture data until canceled', 'default': '1'},
{'name': 'delay', 'kind': {'name': 'POSITIONAL_OR_KEYWORD', 'value': 1}, 'description': 'Time delay in seconds between successive readings; default is 0.', 'default': 'None'},
{
'name': 'per_shot',
'kind': {'name': 'KEYWORD_ONLY', 'value': 3},
'description': 'hook for customizing action of inner loop (messages per step)\nExpected signature ::\n\n   def f(detectors: Iterable[OphydObj]) -> Generator[Msg]:\n       ...',
'default': 'None'
},
{'name': 'md', 'kind': {'name': 'KEYWORD_ONLY', 'value': 3}, 'description': 'metadata', 'default': 'None'}
],
'module': 'bluesky.plans',
'description': 'Take one or more readings from detectors.'
}

And on an annotated version:

```python
>>> from typing import *
>>> from bluesky.protocols import *
>>> def annotated_count(detectors: Sequence[Readable], num: int=1, *, per_shot: Optional[Callable[[Iterable[Readable]], Generator[Msg, None, Any]]]=None, md: Optional[Dict[str, Any]]=None):
...     yield from count(detectors, num, per_shot=per_shot, md=md)
...
>>> ser_plan(annotated_count, existing_devices = {}, existing_plans = {})
{
    'name': 'annotated_count',
    'properties': {'is_generator': True},
    'parameters': [
        {'name': 'detectors', 'kind': {'name': 'POSITIONAL_OR_KEYWORD', 'value': 1}},
        {'name': 'num', 'kind': {'name': 'POSITIONAL_OR_KEYWORD', 'value': 1}, 'annotation': {'type': 'int'}, 'default': '1'},
        {'name': 'per_shot', 'kind': {'name': 'KEYWORD_ONLY', 'value': 3}, 'default': 'None'},
        {'name': 'md', 'kind': {'name': 'KEYWORD_ONLY', 'value': 3}, 'default': 'None'}
    ],
    'module': '__main__'
}

Notes: Some type hints do get interpreted by the function queueserver uses (note the int on "num" in the annotated example)
docstring is also parsed (including extracting descriptions of individual params)
Default value stored as a string, which could be confusing in some contexts
Anywho, that is what you get from "allowed plans", and its origin... additional annotation is available from Dmitri's decorator, but not too interested in that at the moment
Schema is custom, defined  here

Radical answer to serialization problem: literally just str(inspect.signature(plan))
With some agreed upon namespace of objects/types to include typing.* and bluesky.protocols.* (along with builtins)
Downsides: no actual from_string method on inspect.signature, so need to either eval or use ast to deserialize.... this is relatively major, but even so could be dealt with in a reusable way, I think.
Very python specific: UIs built to be e.g. JS native may have trouble
We can go down paths of wanting to be "language agnostic" here, which is the path that led yaq to avro when we wanted schemas... but do we actually care about that?
The string of the signature actually does a surprisingly good job at conveying the complexity of what is a positional vs keyword, required or not, and type in a readable (because it is literally what you see when looking at the code) way that the machinery of python is well tuned to read programatically.
fwiw, jsonschema is all about data, and while a method signature can be thought of as "data", it is not part of the base standard to represent such things, so unless someone else has a more broadly used standard than dimtri's, the moral equivalent of that is what we would get out of jsonschema
pydantic is also about data, not functions, though a little closer to that edge perhaps.
fastapi uses openAPI schema (plus pydantic), which is very much focused on REST apis, so not really applicable here (perhaps it could be adapted, I'm no expert, but largely setting that to the side)


Lifecycle Manager delivery

Description
The Lifecycle Manager is an essential component in the new reconciler architecture. It is following the K8s operator pattern reacting on data changes (defined in CRs).
The Lifecycle manager is responsible for the lifecycle management of Kyma installations (respectively the components configured for a Kyma cluster) and determining the Kyma installation status (aka. cluster status).
Acceptance criteria

[x] #77
[x] #78
[x] #49
[x] Documentation of the architectural design
[ ] Operational requirements are covered

[ ] #79
[x] Exposing analysis and statistic relevant data via Prometheus endpoint
[x] Logging of critical events
[x] Auditlog is integrated
[ ] Operational test was executed and trouble-shooting guides were created

[ ] Threat modelling findings implemented (see https://github.com/kyma-project/lifecycle-manager/issues/144)

[ ] https://github.com/kyma-project/lifecycle-manager/issues/13
[x] https://github.com/kyma-project/lifecycle-manager/issues/146

[ ] Microdelivery process established

Issues
https://github.com/kyma-project/kyma/issues/13759
Appendix

CRD = Customt Resource Definition
CR = Custom Resource
BYOC = Bring your own cluster
SKR = SAP Kyma Runtime

Documentation was recently merged and includes all the architectural and user documentation.
Regarding the audit logs - all the operations are executed via the K8S API server, which is already accounted for in the audit logging configuration.


Plugin doesn't send location updates when the screen goes off

Your Environment

Plugin version: 4.11.4
Platform: Android
Device OS version: 11
Device manufacturer / model: Realme X2
Android Studio version: Chipmunk | 2021.2.1 Patch 1
Capacitor info (npx cap doctor)

@capacitor/core: 4.6.1
@capacitor/android: 4.6.1
@capacitor/cli: 4.6.3
@capacitor/ios: 4.6.1

Plugin config provided to #ready:

{
  stopOnTerminate: false,
  foregroundService: true,
  url,
  headers,
  debug: true,
  xhrFields,
  httpRootProperty: '.',
  locationTemplate,
  distanceFilter: 30,
  elasticityMultiplier: 1.5,
  heartbeatInterval: 60,
  showsBackgroundLocationIndicator: true,
  notification: {
    title: 'Background Tracking Enabled',
    text: 'Your location is being sent to XBE.'
  },
  locationAuthorizationRequest: 'Always',
  backgroundPermissionRationale: {
    title: "Allow App to access to this device's location in the background?",
    message: "In order to track your activity in the background, please enable {backgroundPermissionOptionLabel} location permission\n",
    positiveAction: "Change to {backgroundPermissionOptionLabel}",
    negativeAction: "Cancel"
  }
}

Expected Behavior

Location updates should continue to be sent to our server even when the phone's screen is off.
Actual Behavior

Location updates stop when the phone's screen is turned off.
Steps to Reproduce

I turned on debug mode and am trying to figure out what's happening based on the sound. https://transistorsoft.github.io/capacitor-background-geolocation/interfaces/config.html#debug Here's what's happening:
In the list I can't seem to find the first sound notification that is played when it begins. In the foreground, I get the notification sound location_sample and then location_recorded. After I put it in the background and turned off my screen, it would stay silent and when I turned back the screen on, it picked up again started with the initial notification sound, then the location_sample, then location_recorded. Tried it a few times, it did the same way every time.
Context

I was trying to check if location updates are being sent in the background.
Debug logs

Since the logs were too big to be pasted directly here on github, here's a link to the logs: https://sharetext.me/9ovumqyann

@christocracy Can you please help find what the issue could be? If there's any info I can provide to help with this, let me know.

See https://dontkillmyapp.com

How are you testing?  The device needs to move at least 200 meters before tracking automatically engages.

Sorry for the delay in the response. But yes. We tested with movement > 200 meters as well.

See https://dontkillmyapp.com

so this is something that we can't prevent even if we bought the plugin?  We should really ask the user to exclude the app from battery optimization?

I tried to go out on a bicycle. I stopped to add air to my tire for probably 15mins.  the travel took me 45mins without looking at my phone.  when i arrived home and check the coordinates i got in my server. i noticed that the sending of data stopped where i fixed the tire.   Is this normal? should it resume after 200m?

See api docs Config.stopTimeout.
yes, it’s normal.  Read the wiki “Philosophy of Operation”.

@christocracy - In @jingdaradal's case, I can see why it would stop sending where the tire was fixed. But after he was back moving again, shouldn't the phone Activity recognition have changed state from stationary to moving again after 200m or so? (Providing battery optimisation is switched off for the app and the other hoops that the phone manufacturer may need to not kill it off?)

But after he was back moving again, shouldn't the phone Activity recognition

In a perfect world, yes.  He should provide logs showing what the plugin was doing during that time after stopping.


Exception  "The client_id / audience is invalid!" when using accesstoken

I get an exception when using a token.
This is how I save the token after succesfull logon:
$token = $provider->getAccessToken( 'authorization_code', [
    'code' => $_GET[ 'code' ],
] );
$_SESSION['accessToken'] = $token->getToken() );

This is how I use the stored access token (for debugging I use the token in the session:
$accessToken = $_SESSION['accessToken'];

try {
    $claims = $provider->validateAccessToken($accessToken);
	var_dump($claims);
} catch (Exception $e) {
    var_dump($e);
}

I've checked the token and it is the value created.

ehm. AUD token? where ist that set?

Ah. I assumed that your "I've checked the token and it is the value created" meant that you had decoded the access token.
You can easily check the client id. But to check the audience:

go to https://jwt.io/ , paste your token in there,. and look for the AUD (audience) claim.
If the result is garbage,; de-Base64 the token first using https://www.base64decode.org/

I see thanks, I could see the content. But how is the validateToken supposed to work? I assumed in an API call I can do a quick test to check it the the session is (still) valid.

Since the error message is saying that either or both of the client ID (which you input) and  AUD (which you didn't) are bad, could you please reply with the AUD value - it is not sensitive.
If the token had timed out - they have a very short life of perhaps an hour, or the key signature was bad  - I would hope that the error message would say so.

If I decode the result of getToken and base64_decode it I get something proper but also some garbage at the end
�$@RD_BZC^Pv~9P`�8BZqyt]^D)>ϧEꮉT<`,_#ⴅ3uoR,?orwIjB@dX )RtPtI_C=/Ae-j$tm.\.7j`D:nsʽp垝M[&d%LbȈ
>	wp!yJSKPs\7j$

And json_encode cannot use it.

I cannot tell what the garbage is, but if you are using Graph as the resource provider, the AUD should be https://graph.microsoft.com.
If instead you are using a non-Microsoft package such as PHPMailer to send SMTP mail, the AUD must be https://outlook.office.com/ or authentication will fail; this AUD is forced using a Scope operand

Thanks. but what I see we don't need the results from getToken, only the getResourceOwner data which we will store in a PHP session variable.


Iteration on array in event handler function

Vue version
3.5.12
Link to minimal reproduction
https://github.com/Ericlm/vue-for-of
Steps to reproduce

Create a new Vue project
Create a function that is an event handler in the template (something like @click="() => {}")
If this function receives or creates an array, this array does not always iterate through its items (which are then undefined).

What is expected?
Using a for loop (which could be for-of, for, or forEach) should not make any item "disappear" for the original array.
What is actually happening?
The for loop looks empty, or filled with undefined. For example, with an array of 5 numbers, it can print 5 undefined.

System Info
System:
    OS: macOS 15.0.1
    CPU: (8) arm64 Apple M1
    Memory: 88.38 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 22.6.0 - ~/.asdf/installs/nodejs/22.6.0/bin/node
    npm: 10.8.3 - ~/.asdf/plugins/nodejs/shims/npm
    pnpm: 9.9.0 - ~/.asdf/installs/nodejs/22.6.0/bin/pnpm
  Browsers:
    Chrome: 129.0.6668.103
    Safari: 18.0.1
  npmPackages:
    vue: ^3.5.12 => 3.5.12

Any additional comments?
⚠ It does not occur 100% of the time, but it seems to occur each time the server starts. Sometimes, on HMR, it suddenly works perfectly.

a workaround see Playground


Internal links in Markdown content are broken

The "Formatting text with markdown" link on this page is broken.
Actual:
https://marcusolsson.github.io/modules/formatting-text/
Expected:
https://marcusolsson.github.io/edith/en/modules/formatting-text/

This seems to be preferred way, but breaks other links:
https://github.com/mui-org/material-ui/blob/master/examples/gatsby/src/components/Link.js


[ML][Cherry-pick]fix: Fix include_spark parameter that was erroneonously marked as Optional (#32265)

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-ml


Missing submit buttons in "Navbar" examples

Tackling a failing test in pa11y CI, in #29315
Preview: https://deploy-preview-30464--twbs-bootstrap.netlify.com/docs/4.3/examples/navbars/

it'd probably be slicker if we could have a graphical submit button using the magnifying glass icon or something? not sure how easy that'd be now that we split out bootstrap icons though, admittedly.
otherwise these may feel a bit clunky, visually. not bad, but probably not the slickest of examples.
(and i think for forms with just an input this is not necessarily a big problem, as even on a touch device, users would be on the on-screen keyboard and able to hit enter/submit on that, so not a biggie i think not having a submit button per se there....it's more pa11y flagging something as an error that is arguably more a recommendation/warning)

@patrickhlauke Would you recommend trynig to ignore this specific test on this specific page?
If it's doable, it definetely wortyh a try, I guess.

i would say ignore the whole test. it's opinionated, not directly an accessibility issue per se. i would exclude it from the pa11y run altogether.


Cranelift: avoid debug-mode panics due to wrapping shifts in constant propagation

Fixes #5690

This was more complicated than it looked; Nick and I are working on it.


Open File/Folder Dialog Disappears when Selecting More View Options

VSCode Version: Code 1.10.2 (8076a19fdcab7e1fc1707952d652f0bb6c6db331, 2017-03-08T14:02:52.799Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions: none

Steps to Reproduce:

Open Visual Studio Code
Open the File Dialog with File > Open File... or File > Open Folder...
Go to the Change your view. option and click the down arrow for More options

Maybe a Windows 7 Classic Theme issue?.. please see attached:

Since this issue is rather old, I am closing this and ask anyone to open a new issue if this is still happening.


Doubled value for "reply_count" in messageReplyInfo

In an ordinary supergroup chat, if I execute "getMessage" method directly I got "reply_count" doubled of actual value.
getMessage chat_id=-100xxx message_id=366371405824
==>
{"@type": "message", "id":<PHONE_NUMBER>24, ... "interaction_info": { .. "reply_info": { .. "reply_count": 8 ..}}}

However If I do:
getMessageThread chat_id=-100xxx message_id=366371405824
==>
{... messages:[{"@type": "message", "id":<PHONE_NUMBER>24, ... "interaction_info": { .. "reply_info": { .. "reply_count": 4 ..}}}]}

After getMessageThread execution getMessage also starts to return correct value for "reply_count".
Is this a bug or a feature?
Thanks

I see this behaviour only in some supegroups.  In most supergroups "reply_count" value is correct out of box, without need for getMessageThread call

This happens when a lot of old messages are received. Server returns current, up-to-date value of reply_count and TDLib increments it again when receiving new replies to the message. It is not possible to fix this reliably without changes of the server API, but for the most cases it has already been fixed in the private TDLib version and will work better in TDLib 1.7.1.

Should be fixed in master.


Fix comments in return statement argument

I couldn't figure out how to add a commit to #657 so creating another pull request for it. I think that we can just check if there's a leading comment and if that's the case then put parenthesis. This should work better than checking the source location.
Fixes #611

cc @yamafaktory, if you like this approach, can you patch this in your PR and I'll merge it :)

@vjeux all done!


Resync workinggroup

Align this working group master branch with the kubernetes-sigs/scheduler-pllugin master branch.
This step is a prerequisite before creating a release flow

do we want a lane for sanity check post-merge to run at least unit tests?


replace uses of old grid in website and documentation

This was tedious, but important.
closes #2927

@jeeyun The challenge is writing a schematic that does this correctly, because these classes are found in many places (could be in a template (html or ts file), or a host binding, or a ngClass binding, or other places). I ultimately didn't feel confident we could do that correctly.

Makes sense. Thanks for the feedback.


add upbound provider when using eksctl bootstrap

What does this PR do?
add upbound provider when using eksctl bootstrap
Motivation
Resolves #93
More

[x] Yes, I have tested the PR using my local account setup  (Provide any test evidence report under Additional Notes)

[x] Yes, I have added a new example under examples to support my PR

[x] Yes, I have updated the docs for this feature

[x] Yes, I have linked to an issue or feature request (applicable to PRs that solves a bug or a feature request)

Note:

Not all the PRs require examples and docs
We prefer small, well tested pull requests. Please ensure your pull requests are self-contained, and commits are squashed

For Moderators

[x] E2E Test successfully complete before merge?

Additional Notes

@candonov PTAL when you have chance


[feature request] C++ and Python support negative acknowledgment backoff

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
PIP-106
Solution
Refer java client: https://github.com/apache/pulsar/pull/12566
Alternatives
No response
Anything else?
If the redeliveryCount is greater than the DLQ configuration, it needs to be sent to the DLQ.
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Is this not yet supported?


How to get the evaluation results of camera-only 3D detection?

I try to submit my prediction results of validation set following https://github.com/waymo-research/waymo-open-dataset/blob/master/docs/quick_start.md/ to the website. But there are still not any results can be found. What should I do?

What's your submission email?

Addressed in #491. Please close this issue, Thanks!


VK_EXT_line_rasterization support

Hello!
Is there any possibility to get support for VK_EXT_line_rasterization in the future? I don't know if there is anything similar in Metal.

Native line rendering in Metal is incredibly primitive (excuse the pun).
What features of VK_EXT_line_rasterization are you looking for? Stippling? Coverage options (Bresenham, Smooth, etc)?

If it was possible to have VK_LINE_RASTERIZATION_MODE_RECTANGULAR_SMOOTH_EXT implemented sometime in the future. But I understand if this is not really a priority, there must be more important extensions 😅
I also find this very funny in specs:
The details of the rasterization rules and area calculation are left intentionally vague, to allow implementations to generate coverage and values that are aesthetically pleasing.


CAMEL-20838: avoid causing an NPE during shutdown or initialization if the context manager is not available

@orpiske This works for me, but I obviously get the warning "Unable to set the global store for [...] because the context manager is not available" for every test. Isn't an idea to just move lines 149-150 inside the if?

@orpiske ... As the execution is always beforeAll -> beforeEach, the contextManager will always be null unless the perClass annotation is present, or am I missing something?

That's correct.

Updated #14641 with this change.

Thanks. I will also review that, run a full test on it and we'll go with either one of them (likely yours - after I review it).

Merged #14641 instead. Let's close.


name functions via the displayName property

There is a convention used by at least the following debuggers to allow anonymous functions to be named by the source code:

Web Inspector
FireBug
weinre

The convention is that a function which otherwise doesn't have a name property which is a string, will be checked to see if it has a displayName property, and that will be used in lieu of the name.  The convention was originally added to Web Inspector by the Objective-J developers.
I would like this support added to CoffeeScript as a new option of the coffee compiler.
There's less need for this today, with the JS engines doing name inferencing for anonymous functions, but not all JS engines support this.  In addition, the JS engines that weinre supports (eg, on mobile devices) are typically older than their desktop brethren, and are less likely to have that support.
Even with name inferencing, some folks might prefer a slightly cleaner look.
Proposal:

new command line option -f which will add the displayName property to every function CoffeeScript produces, except for constructors which are already named.

I see 4 forms of functions, which would presumably have different naming styles:

static methods of a class: (className)::(methodName)
instance methods of a class: (className).(methodName)
functions in a class: (className):(functionName)
functions not in a class: (functionName)

I'm not married to the choice separators in the names, that was just my initial thought.
Methods in a class should be babycakes; functions will be harder, and I don't have a sense for how much harder.  Certainly, there are cases where there is no obvious name, and those functions should be left anonymous.
Here's an example.
CoffeeScript:
class X
  @a: () -> a
  b: () -> b
  c = () -> c

d = () -> d

JavaScript produced today:
var X, d;
X = (function() {
  var c;
  function X() {}
  X.a = function() {
    return a;
  };
  X.prototype.b = function() {
    return b;
  };
  c = function() {
    return c;
  };
  return X;
})();
d = function() {
  return d;
};

JavaScript produced tomorrow with the -f option:
var X, d;
X = (function() {
  var c;
  function X() {}
  X.a = function() {
    return a;
  }; X.a.displayName = "X::a"
  X.prototype.b = function() {
    return b;
  }; X.prototype.b.displayName = "X.b"
  c = function() {
    return c;
  }; c.displayName = "X:c"
  return X;
})();
d = function() {
  return d;
}; d.displayName = "d"

I'd be happy to work a patch for this, but wanted to see if there was a hope that a contribution for this new function would be accepted, before starting the work.

You hit the nail on the head as to why this isn't desirable ...

There's less need for this today, with the JS engines doing name inferencing
for anonymous functions [...]

The displayName property was a temporary hack, needed because name inferencing wasn't available at the time. In general, inferenced (dynamically bound) names are far superior to static names defined at the same time the function is. For example:
f(x){ return this.total + x; }

ShoppingBag.prototype.add = f;

Counter.prototype.increment = f;

... it's totally fantastic that when debugging, you'll see the two proper different names for the function, instead of f.
Feel free to add displayName in your fork -- it should be fairly easy to add -- but it shouldn't be available on the main line.


PULL_REQUEST_TEMPLATE.md is empty

Under the .github folder, the PULL_REQUEST_TEMPLATE.md only has "# Fixes" written on it. Was this never written? or was it deleted?
I am looking for what version/configuration of JS and solc linters to set up on my environment to start contributing.
The PULL_REQUEST_TEMPLATE.md file is mentioned (under typical workflow , step 4) but not linked on the CONTRIBUTING.md file.
Thanks

Oh, interesting, thanks! We should add that information to the template. It may have been deleted at some point, we used to have a larger one (that was not very useful and was therefore shortened).
Regarding setup, having Node ^v8.0, and running both npm run lint and npm test should suffice. npm install will install all required dependencies to run both of these commands: no other tools are needed.

Awesome. Thanks for the clarification @nventuro.

Oh, actually the file does have a lot more content, but it's written in HTML comment tags, so it doesn't show up in the GitHub preview. It definitely does show up in the Pull Request text field, though!
https://raw.githubusercontent.com/OpenZeppelin/openzeppelin-solidity/master/.github/PULL_REQUEST_TEMPLATE.md


bitbucket-snippet-runner 0.1.0

New Pull Request Checklist
Please mark the points which you did / accept.

[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x]  I read and accept the Abandoned Step policy

Thanks for the PR @nhammond101 , we'll review this ASAP on Monday!

Hey @nhammond101 !
LGTM 🚀
Thanks for sharing! 🎉


chore: deativate LNv2 inter-federation test

LNv2 inter federation test is flaky: https://github.com/fedimint/fedimint/issues/5944
Seems to be an issue with CLN crashing during the test for some reason. I have found it difficult to debug as I cannot reproduce locally and the CI runs have not had enough logs to debug properly.
To unblock CI, this PR deactivates the test from CI.

Thanks for putting this up! I was just about to do the same :slightly_smiling_face:


[WIP/MRG] Backend/feat/no password second email confirmation added test

Adding a test to check that the tokens successfully expire. Not super thrilled with this solution:

I would have preferred to mock func.now(), but couldn't get it to work
Would prefer to be able to use now() instead of couchers.utils.now() (and not have to import couchers), but again, couldn't get it to work.

Creating this as a branch to see if anyone else knows how to fix either of the above two issues
Backend checklist

[x] Formatted my code by running isort . && black . in app/backend
[x] Added tests for any new code or added a regression test if fixing a bug
[x] All tests pass
[x] Run the backend locally and it works
[x] Added migrations if there are any database changes, rebased onto develop if necessary for linear migration history

I seem to remember mocking now in tests for calculating the user's age.

@lucaslcode Did you mean in test_models.py? It looks like you mocked date, not now, but maybe I'm missing something?


Linter

What kind of change does this PR introduce? (Bug fix, feature,
docs update, ...)

Bug fix

What is the current behavior? (You can also link to an open issue
here)

The lint is giving errors

What is the new behavior (if this is a feature change)?

Fix the lint issues

Does this PR introduce a breaking change? (What changes might
users need to make in their application due to this PR?)

No

Updated!


Release Google.Cloud.Batch.V1 version 2.0.0

Changes in this release:
BREAKING CHANGE
BatchServiceClient no longer exposes an IAMPolicyClient. This was only exposed unintentionally, and has never worked. However, this is still a breaking change as application code may have referred to it.
New features

Added StatusEvent.task_state (commit 25463ba)

Documentation improvements

Updated comments (commit 25463ba)

Created release for Google.Cloud.Batch.V1-2.0.0


Auto Incrementing Ids

Is it possible to include auto-incrementing Ids? by using an attribute on the POCO for instance.
That would be awesome!!!
Awesome Project By The Way

Hi Luis, I have plans for that. I'm working on dev branch before new releases.
Auto Id will be using not only in POCO, but in BsonDocuments too. So, if you use shell, _id will be created. I need implement ObjectId too...

Hi Luis, take a look on this commit on dev branch: https://github.com/mbdavid/LiteDB/commit/bf0cde93828c3032b1905264aac7bbc17cbe5eaa

Good Stuff!!! Thanks


Migrating supervised contrastive learning example to Keras 3

This PR changes the Supervised Contrastive Learning example to keras 3.0 [TF-Only Example] as requested in https://github.com/keras-team/keras-cv/issues/2211.
For example, here is the notebook link provided:
https://colab.research.google.com/drive/1QGiil-RpO55UNESBkilNtYETuX3B_4ZF?usp=sharing
cc: @divyashreepathihalli @fchollet
The following describes the Git difference for the changed files:
Changes:
diff --git a/examples/vision/supervised-contrastive-learning.py b/examples/vision/supervised-contrastive-learning.py
index 4803e671..6b45d568 100644
--- a/examples/vision/supervised-contrastive-learning.py
+++ b/examples/vision/supervised-contrastive-learning.py
@@ -20,22 +20,20 @@ Learning is performed in two phases:
 that representations of images in the same class will be more similar compared to
 representations of images in different classes.
 2. Training a classifier on top of the frozen encoder.
+"""

-Note that this example requires [TensorFlow Addons](https://www.tensorflow.org/addons),
-which you can install using the following command:
-
-```python
-pip install tensorflow-addons
-```
-
+"""
 ## Setup
 """

+import os
+
+os.environ["KERAS_BACKEND"] = "tensorflow"
+import keras
+from keras import ops
+from keras import layers
 import tensorflow as tf
-import tensorflow_addons as tfa
 import numpy as np
-from tensorflow import keras
-from tensorflow.keras import layers

 """
 ## Prepare the data
@@ -159,6 +157,19 @@ softmax are optimized.
 """

+def npairs_loss(y_true, y_pred):
+    y_pred = ops.convert_to_tensor(y_pred)
+    y_true = ops.cast(y_true, dtype="uint32")
+
+    # Expand to [batch_size, 1]
+    y_true = ops.expand_dims(y_true, -1)
+    y_true = ops.cast(ops.equal(y_true, ops.transpose(y_true)), dtype="uint32")
+    y_true /= ops.sum(y_true, 1, keepdims=True)
+
+    loss = tf.nn.softmax_cross_entropy_with_logits(logits=y_pred, labels=y_true)
+    return ops.mean(loss)
+
+
 class SupervisedContrastiveLoss(keras.losses.Loss):
     def __init__(self, temperature=1, name=None):
         super().__init__(name=name)
@@ -174,7 +185,7 @@ class SupervisedContrastiveLoss(keras.losses.Loss):
             ),
             self.temperature,
         )
-        return tfa.losses.npairs_loss(tf.squeeze(labels), logits)
+        return npairs_loss(ops.squeeze(labels), logits)

 def add_projection_head(encoder):
(END)

So, the updated code is working fine with the TensorFlow backend. With other backends, like Jax, I tested, it is giving errors like TracerArrayConversionError: The numpy.ndarray conversion method __array__() was called on traced array with shape float32[265,128].. Although there are two instances of the tf method (not tf.data). The changes I made were basically removing the TensorFlow addons. Tensorflow addons are used for implementing the npair loss. So, I made a function implementation by following this code. And it also worked with the entire example code successfully.

@sitamgithub-MSIT Thank you for the PR! You will need to replace all the Tensorflow ops with Keras ops - here is a list of all Keras ops. Here is a guide for migration

The two only instances where TF methods are used are:
def npairs_loss(y_true, y_pred):

    y_pred = ops.convert_to_tensor(y_pred)
    y_true = ops.cast(y_true, dtype="uint32")

    # Expand to [batch_size, 1]
    y_true = ops.expand_dims(y_true, -1)
    y_true = ops.cast(ops.equal(y_true, ops.transpose(y_true)), dtype="uint32")
    y_true /= ops.sum(y_true, 1, keepdims=True)

    loss = tf.nn.softmax_cross_entropy_with_logits(logits=y_pred, labels=y_true)

    return ops.mean(loss)

Now I know loss = tf.nn.softmax_cross_entropy_with_logits(logits=y_pred, labels=y_true) can be substituted with keras.ops as described here. But whenever I am doing this, an error shows up: ValueError: Cannot take the length of shape with unknown rank..
Another instance is this tf.math.l2_normalize(feature_vectors, axis=1). So, I need some help in only these two instances to deal with.

here

do you have the repro for this error?

here

do you have the repro for this ValueError?
For l2_normalize - please use this normalize function and set the order to 2.

Thanks for the help! I was actually able to omit both TF instances. But other backends are showing errors. This is the modified code that worked well with Tensorflow.
def npairs_loss(y_true, y_pred):

    y_pred = ops.convert_to_tensor(y_pred)
    y_true = ops.cast(y_true, dtype="uint32")

    # Expand to [batch_size, 1]
    y_true = ops.expand_dims(y_true, -1)
    y_true = ops.cast(ops.equal(y_true, ops.transpose(y_true)), dtype="uint32")
    y_true /= ops.sum(y_true, 1, keepdims=True)

    y_true.set_shape([None, None])
    loss = ops.categorical_crossentropy(y_pred, y_true, from_logits=True, axis=-1)

    return ops.mean(loss)

But I see this set_shape is not going to work with other backends showing attribute errors. For example, jax AttributeError: 'ArrayImpl' object has no attribute 'set_shape' is showing this.

I think you can just omit y_true.set_shape([None, None]).

I think you can just omit y_true.set_shape([None, None]).

Then it is giving this error with the TensorFlow backend ValueError: Cannot take the length of shape with unknown rank.


Help required

Hello @m-Ryan, during the easy email learning process I accidentally uploaded file with private information to your cloudinary cloud. Could you please clarify can I remove this file without disturbing you? I tried to find solution but looks like to call remove method I need to provide api key.
Thank you

Please provide file links and I will delete it. You can send an email to me.

I sent the link on<EMAIL_ADDRESS>Thank you

Done. Please check. @dmitry-osin

@m-Ryan it is still available by link that I've shared with you via email

It may be due to cache, you can use incognito mode to check.

Yes, looks like it has been cached by chrome. Thank you 🙏


Stuck in "Exporting items" with 7.0.0-beta.71+03d0ea436

Debug log ID
D1678910747
What happened?
Debug log cannot be generated because the BBT debug log just wont popup. I think the new version is not supported yet.

Same here

See #2829

Please subscribe to #2829 if you want to track progress

Right, it seems to be the same problem.
Luckily Linux version works fine so far.

If it had broken on Linux I would have known sooner, because my test suite runs on linux.


Where is NDI in New Teams?

This page says NDI & SDI out support is coming in November.  Given the context that the next thing mentioned is January (which also mentions no year) can you confirm when NDI out will be available in New Teams desktop for windows and Mac?

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

ID: 442f8c6b-95ba-a09e-639d-b9dd141198f6
Version Independent ID: 442f8c6b-95ba-a09e-639d-b9dd141198f6
Content: Known issues in the new Microsoft Teams desktop client - Microsoft Teams
Content Source: Teams/new-teams-known-issues.md
Service: msteams
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Always use --startup-file=no

Codecov Report

Merging #16 into master will decrease coverage by 4.20%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
- Coverage   92.24%   88.03%   -4.21%
==========================================
  Files           3        3
  Lines         116      117       +1
==========================================
- Hits          107      103       -4
- Misses          9       14       +5

Impacted Files
Coverage Δ

src/foreach.jl
89.24% <0.00%> (-5.38%)
:arrow_down:

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 64c47ea...b93ae4b. Read the comment docs.

Codecov Report

Merging #16 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #16      +/-   ##
=========================================
+ Coverage   92.24%   92.3%   +0.06%
=========================================
  Files           3       3
  Lines         116     117       +1
=========================================
+ Hits          107     108       +1
  Misses          9       9

Impacted Files
Coverage Δ

src/PerformanceTestTools.jl
88.88% <100%> (+0.65%)
:arrow_up:

src/foreach.jl
94.62% <0%> (ø)
: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 64c47ea...b93ae4b. Read the comment docs.


integration_time plus 1 second (fixed) blocked reliable measures

https://github.com/maxlklaxl/python-tsl2591/blob/290ef844d8a939eb6674f241f0b17af8667764c0/tsl2591/read_tsl.py#L158
Typo here: () missing
time.sleep(0.120self.integration_time+1)  #  takes minimal 1 second + 0.120self.integration_time
You want:
time.sleep(0.120*(self.integration_time+1))
Fastest working is:
time.sleep(0.105+0.100*self.integration)

Seconded! I use a local copy of this helpful repository and fixed it there but never reported here, sorry...
The initial clue for that comes form the official Adafruit driver and datasheet: https://github.com/adafruit/Adafruit_TSL2591_Library/blob/master/Adafruit_TSL2591.cpp#L316-L320 in combination with https://github.com/adafruit/Adafruit_TSL2591_Library/blob/master/Adafruit_TSL2591.h#L84


Translation processing problem

When you have a sentence and dots are found in the middle, the sentence cannot be completed and only the first part is translated, ignoring the last portion after dots. for example
The officers and employees of the Bank, who are not local nationals of the Kingdom of ................... shall be exempt from customs duties and other levies, prohibitions and restrictions on the importation of motor vehicles and spare parts thereof, and household effects, equipment and furniture.
The result comes only for the first part until Kingdom of

Hello Khalid
Are you translating into Arabic by any chance?
I wouldn't be surprised if this has something to do with right to left languages but I'm only guessing of course
The thing is that when I tested what you said in one of my language pairs (Swedish to English), Opus CAT translated everything (see attached file)

So, it seems that the problem is in the language you are translating into, but this should not happen since the tool is counting the source words and compare them to the target words. It may be a bug and needs to be fixed.
Thanks for your guressing

Tommi,
Would you try this sentence and see the result:
1996 ................... among certain African states and international organizations;

Here is the database

That looks like a fine-tuned model, so it's possible that this caused by the fine-tuning process. Since the data used for fine-tuning is generally very domain-specific, it may cause performance to degrade with source texts that don't belong to the fine-tuning domain (such as these kinds of texts where a series of periods is used as placeholder).
How much data did you use to fine-tune the model with, and what sort of data was it? Another complicating factor is that the Arabic models are multilingual models, i.e. they support multiple variants of Arabic, which might affect fine-tuning.

Over one million segments

Most of the data is almost in the main domain

Ok, that's a lot of data. It does sound like the problem with the repeated periods is caused by the fine-tuning. If there are other errors in the translations besides the problem with the repeated periods, I would advise fine-tuning with smaller, more targeted set of segments.
If the model translates OK otherwise, it's also possible to use a pre-edit rule to edit those problematic sentences automatically before they are translated. For instance, you could use a rule like this:

This rule would truncate all series of repeated periods to five periods, which might be easier for a MT model to handle.


docs: describe meta_generator option

Fixes https://github.com/hexojs/site/issues/1019
https://github.com/hexojs/hexo/pull/3653

@tomap
This feature for hexo v4 (Already merged)


Thank you - Works great for 10.0.19041.84

Hello Sebaxakerhtc - thank you for posting your solution.  It appears that you may be one of the last repositories where folks like me can find updated offsets - if MS shuts you down will your code continue to pull down new ini files with new offsets?  I don't want to see RDPWrap disappear.  Thank you again!

Hi.
I'm ready for that 😁
I will edit file "autoupdate.bat" to recieve and apply new "rdpwrap.ini"
from my own nextcloud ;)
пт, 29 мая 2020 г., 17:41 CharlieF2<EMAIL_ADDRESS>
Hello Sebaxakerhtc - thank you for posting your solution. It appears that
you may be one of the last repositories where folks like me can find
updated offsets - if MS shuts you down will your code continue to pull down
new ini files with new offsets? I don't want to see RDPWrap disappear.
Thank you again!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sebaxakerhtc/rdpwrap/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHZDR4TDUBDTG2JVRWPNPD3RT7CR3ANCNFSM4NODES6A
.

Hello Sebaxakerhtc - thank you for posting your solution. It appears that you may be one of the last repositories where folks like me can find updated offsets - if MS shuts you down will your code continue to pull down new ini files with new offsets? I don't want to see RDPWrap disappear. Thank you again!
redownload new mod for a "plan B" :)


iOS -> iOS not working

Both iOS -> AppRTC and AppRTC -> AppRTC are working fine. But can't seem to get an established connection between 2 iOS devices working.
Connection doesn't seem to get established and continuous 'Waiting for another person to join the room' message appears.
Did someone else get this working?

It seems that rooms can only be created from the web app, maybe due the Origin header. Nothing we can do. Anyhow this is just a demo.


Update oms-windows.md

The agent version of the latest entry is incorrect: 10.30.18040 and not 10.20.18038

@josecons-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


SMARTS for hexa-coordinated compounds

Are we able to define SMARTS for atoms attached to a hexa-coordinated atom with the SMARTS functionality we currently have?  The compound of interest in this case is PF6.  If all angles within the molecule are to be considered, then three fluorine atom types will be necessary.  All Fx-P-Fx angles should be 180 degrees and all Fx-P-Fy angles should be 90 degrees.  However, because the atom types are dependent on orientation rather than bonding, I'm not sure we're able to assign atom types with our current SMARTS functionality.
One possible solution to this involves adding the NOT operator "!" into our SMARTS logic.  I'm then envisioning an XML that would look something like this:
<Type name="P" class="P" element="P"  mass="30.9738"  def="P" desc="phosphorous"/>
<Type name="F1" class="F1" element="F"  mass="18.9984"  def="FP([!%F1])([!%F1])([!%F1])([!%F1])F" desc="fluorine type 1"/>
<Type name="F2" class="F2" element="F"  mass="18.9984"  def="FP([!%F1;!%F2])([!%F1;!%F2])(F1)(F1)F" desc="fluorine type 2" overrides="F1"/>
<Type name="F3" class="F3" element="F"  mass="18.9984"  def="FP(F1)(F1)(F2)(F2)F" desc="fluorine type 3" overrides="F1,F2"/>

This should atom type two fluorines each as F1, F2, and F3.  The geometry wouldn't be guaranteed, but a simple energy minimization would fix that.

I'm just spectating in general, since I'm not currently involved in this project (but am an interested onlooker). However, I can't resist chiming in on this one. I'd encourage you guys to get in as much of SMARTS as you can - which means you should be able to use "not", when appropriate.
I'm not totally sure I understand the problem here, though -- but just to raise the question, is this a problem that would be more easily solved with SMIRKS, where you have the ability to tag an atom to refer to it? e.g. [#8:1]~[#6:2]~[#9:3] would be a oxygen-carbon-fluorine, where the carbon is tagged as "atom 1" and the fluorine as "atom 3".

My explanation of the problem was not very clear, but basically compounds featuring a hexa-coordinated atom are an interesting edge case where two different angles are present between atom triplets that would normally have the same atom types (in the case of PF6, there are F-P-F angles of both 180 degrees and 90 degrees).  If all fluorines are given the same atom type (as would normally be the case due to all fluorines featuring the same connectivity/local chemistry), Foyer would be unable to choose which F-P-F angle to assign.  The workaround is to define three fluorine atom types (F1, F2, F3) such that F-P-F angles of like fluorine types (e.g. F1-P-F1) are 180 degrees, and those of unlike fluorine types (e.g. F1-P-F2) are 90 degrees.  However, because there are no differences in the local chemistry for each fluorine, we run into a problem with how to define the SMARTS string for each of the F1, F2, and F3 atom types.
I think adding the "not" operator into Foyer and defining the SMARTS as I showed above will get around this problem.  The plan to add this operator (and more of the SMARTS functionality) into Foyer has already been in the works.
Having the ability to assign a tag to individual atoms as you've suggested, following the SMIRKS format would also solve this problem I think.  The issue here is where the tag would live within the data structure.  This could possibly be added as an attribute to an mBuild Compound, but internally would need to be able to be passed on to both ParmEd Structures and OpenMM Topologies.

I'd encourage you guys to get in as much of SMARTS as you can - which means you should be able to use "not", when appropriate.

That's absolutely high priority. It's hard to tell people that we're SMARTS based but you can only use a tiny subset :)

is this a problem that would be more easily solved with SMIRKS, where you have the ability to tag an atom to refer to it?

I don't think so. As @summeraz explained, the weirdness here comes from the fact that the force field angle types are written in a geometry dependent way - a pure topology doesn't contain enough information to directly resolve whether a given F-C-F angle is supposed to be the 90 or 180 degree term.

@summeraz I think I understand what you were saying the other day now. I suspect your idea could work. I am curious how trivially an energy minimization would resolve this situation though. We should be getting more SMARTS functionality built in early this week so we can give it a whirl.

Ah, OK, I understand the issue now. You just need to end up with three fluorines of each type (and yes, the minimization should sort it out, er, hopefully. One would need to check.) So it seems like you'd want to just come in at the first pass and make them all (type 1) and then make type 2 be any fluorine which is connected to a P which already has at least three of (type 1). There should be a variety of well-formed strings which will let you do this. But I agree, you'll likely need the "not" operator.
This is a bit odd, though, as to handle this you need to make your SMARTS strings reference atom types, not chemistry, right?? To me this seems like the language is telling you that you're committing an "unnatural act".
I wonder if there is a way to get the geometry you want without doing this. (Though, I suppose your perspective is that you're trying to put the existing force field into this format, not find a more natural way of getting the same geometry even if it means modifying the force field...)

Though, I suppose your perspective is that you're trying to put the existing force field into this format, not find a more natural way of getting the same geometry even if it means modifying the force field...

This is the crux of the problem. There are a few weird edge cases in force fields because they were not developed with considerations like this in mind. Something that OpenFF will hopefully solve but doesn't immediately allow us to use existing force fields :confused:


Enhance hub servlet ListAllNodes

ListAllNodes will now return application/json when the header "Accept: application/json" is present
Improve ListAllNodes text/html view to;

Display more node information
Display information which is applicable to nodes which use
SeLionRemoteProxy such as uniqueSessionCount and more
Optionally ping the node for /status when the query param ?pingNodes
is present
Render view from embedded JSON output -- which is the same output
ListAllNodes returns if the request is for a JSON only response

Misc changes including;

Remove RESOURCE_PAGE_FILE related constants from SeLionGridConstants
Remove tabs from view.css
SeLionRemoteProxy now pings the node at initialization to determine
if it supports SeLion servlets that are needed for key capabilities
such as 'auto upgrade' and 'restart'. Methods added to get the
result of the findings.
Other misc informational / state related methods added to SeLionRemoteProxy
Fix potential NPE in AuthenticationHelper which occurred client
session is active but server was restarted.
Introduce respondAsHtmlUsingJsonAndTemplateWithHttpStatus in
ServletHelper.java
Introduce new resource path for servlets /com/paypal/selion/* which
is more inline with selenium-server's pattern -- all SeLion servlets
to follow suit eventually

Screenshot of updated ListAllNodes responding with text/html

Example of updated ListAllNodes servlet responding with application/json
[{
    "logsLocation": "http://<IP_ADDRESS>:5555/extra/LogServlet",
    "isBusy": false,
    "percentResourceUsage": 0.0,
    "totalUsed": 0,
    "isShuttingDown": false,
    "totalSessionsComplete": 0,
    "totalSessionsStarted": 0,
    "uptimeInMinutes": 7,
    "configuration": {
        "port": 5555,
        "nodeConfig": "/Users/dsimmons/.selion/config/nodeConfig.json",
        "hubConfig": "/Users/dsimmons/.selion/config/hubConfig.json",
        "registerCycle": 300000,
        "newSessionWaitTimeout": -1,
        "prioritizer": null,
        "remoteHost": "http://<IP_ADDRESS>:5555",
        "throwOnCapabilityNotPresent": true,
        "proxy": "com.paypal.selion.proxy.SeLionRemoteProxy",
        "maxSession": 20,
        "role": "node",
        "uniqueSessionCount": 50,
        "host": "<IP_ADDRESS>",
        "servlets": "com.paypal.selion.node.servlets.LogServlet,com.paypal.selion.node.servlets.NodeAutoUpgradeServlet,com.paypal.selion.node.servlets.NodeForceRestartServlet",
        "cleanUpCycle": 30000,
        "browserTimeout": 180000,
        "selionConfig": "/Users/dsimmons/.selion/config/SeLionConfig.json",
        "hubHost": "<IP_ADDRESS>",
        "capabilityMatcher": "com.paypal.selion.grid.matchers.MobileCapabilityMatcher",
        "url": "http://<IP_ADDRESS>:5555",
        "nodeRecycleThreadWaitTimeout": 0,
        "register": true,
        "nodePolling": 5000,
        "jettyMaxThreads": -1,
        "hubPort": 4444,
        "timeout": 120000
    },
    "status": "online",
    "version": "2.47.1",
    "os": "Mac OS X 10.10.5",
    "slotUsage": {
        "firefox": {
            "used": 0,
            "percentUsed": 0,
            "maxInstances": 15
        },
        "chrome": {
            "used": 0,
            "percentUsed": 0,
            "maxInstances": 10
        },
        "phantomjs": {
            "used": 0,
            "percentUsed": 0,
            "maxInstances": 10
        }
    }
}]

@sebady @sundaramrajendran Please take a look. Note: I've already begun to build on the approach this PR takes for rendering html and where the resource files are stored in an effort to update the other servlets we have to follow suit. I want to rip out the html which is mixed-in with our current servlet java code. Thus, if you two prefer we can merge all changes as one or individually with this one first.

closing.. will be included with #207


Autoload scrapers using Adapter pattern

Fixes #233, #265, #271, #273
Checklist

[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream master branch.
[x] I have added necessary documentation (if appropriate)

Changes proposed in this pull request:

Instantiate the requested engine on demand instead of instantiating all at startup time
Treat Scrapers class as an Adapter interface (#233) and autoload all subclasses into a dict.  This means that the proper names are being used (#273) which eliminates the single character lookup issues and also facilitates the addition of new scrapers with just a change to index.html and adding the name to test_server.py.
Yandex now loads (#271) but needs work so it is removed from one of the tests.
Parsijoo is added to the testing.

Rebased yet again

@cclauss Woow!! Good work..will review in a day or two

Can we have a deployment link for this particular changes.
Since we load the scrapers by reading the directories on the fly, maybe there will be some delay in getting results no ?
What do you think @vaibhavsingh97 ?

I like the idea of loading scrapers directly from directory so we no longer need to keep typing their names 👍
But with the current implementation, directory listing is done on every request, which seems unnecessary since once deployed, code won't change so scrapers will be the same as before.
So we can just save the list of parsers once and then use the same on each request.

Thats a legit point @AnshulMalik
@cclauss please take a note of this :smile:

Lets say we added the names of scrapers in os env  on first load. Will that help @AnshulMalik
One possible solution I can think of.

We can have the result of get_scrapers stored in a global variable @raju249

That works :+1:

scrapers.get_scrapers() can be run consistently more than 2,300 times in one second.
import timeit
from scrapers.scraper import get_scrapers
print(timeit.repeat(get_scrapers, repeat=3, number=2300))

But I will put the results scrapers.get_scrapers() (with each scraper instantiated) into a module variable at startup time.  The downside of that approach is that the server will need to be restarted to recognize that a new scraper has been added (or deleted) but I think that is a fair compromise.
On the pros and cons of instantiating the scrapers at startup time:

Pros:

Less time on every search
Simplifies the caller's logic

Cons:

Server must be rebooted to detect added (or deleted) scrapers
More time at startup
Any memory leaks in a scraper persist for the entire run time of the server
Scrapers are probably not reenterant making threading, asyncio trickier

Despite the cons, I believe that we should still instantiate all scrapers at startup time


Page titles are not translated on switch

For example, switching between translations on /#!/project/ipvision will keep the first value.

Fixed for the page display by adding a translate directive, the <title> still not working properly.

@danmindru this seems to be working now for me.


Api/Website down

Seems like the API/Website are down since yesterday morning.
@Mechazawa hopefully you just need to kick the server?

It's seems more serious than that..
https://github.com/Pushjet/Pushjet-Android/issues/34

Ouch, thanks for pointing out the other thread. I'll follow that one.


graph display 'ignores' dates with no data

Ian, If you select and NI region and a date range starting where there is no data and going into a place where there is f.ex Jan 2000 - jan 2016 graph starts at 2005 is this intentional?
It confused me slightly when I first looked at it as I thought the program had bugged out - don't think its major and certainly doesn't need looking at until after go live and then only if we have queries on it

The graph axis extent is current set from the data, not the dates on the form. I could change that, but at the moment you get the range that's in the data.
If we do change it, one effect will be to have a blank margin on the right of the graph. The default 'to' date in the form is 'today', and there's never a case when the latest published data goes all the way up to today.

Thanks Ian - Solves my question - probably better to see if it causes problems to the users before looking further at this.


ERROR: Manifest merger failed with multiple errors, see logs

error: failed linking file resources.    activity_code_tool.xml:20: error: resource dimen/dp_55           sos

When integrating this plug-in, if you use support.v4 or support.v7, change the support library to Android x, otherwise you will not be able to merge the list. If you change it in Java code, remember to change it in the layout file

最新版本已迁移至 AndroidX, 更新到最新版本试试

TangYuLi<EMAIL_ADDRESS>於 2020年1月16日 週四 10:39 寫道：

+1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Tamsiree/RxTool/issues/233?email_source=notifications&email_token=AC7ZY2GAWHHPGCFA7JDBYGLQ57CGFA5CNFSM4KBJRHKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJCRW6Q#issuecomment-574954362,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC7ZY2CFYA5KOZAINAYJBBTQ57CGFANCNFSM4KBJRHKA
.


fix attacher() on WorldAttachment

I just stumbled accross some 500s on my website when linking events to stuff. This change seems to have fixed that! Was told to open a PR if I wanted to get the fix reviewed o/

Ah yes, thank you!


Hostile Mobs Won't Go Through Portals

Hi, a server I assist on is having an issue with nether portals in that passive mobs & items can pass through portals however hostile mobs just vanish.
The config is set up as follows:
bounceback: true
teleport_entities: true
send_no_destination_message: true
send_disabled_portal_message: true
worlds:
testing_world:
portalgoesto:
NETHER: testing_world_hell
testing_world_hell:
portalgoesto:
NETHER: testing_world
HermitVeterans:
portalgoesto:
NETHER: hermitveterans_netherworld
hermitveterans_netherworld:
portalgoesto:
NETHER: HermitVeterans
The server is running on Purpur 1171 (the latest and last 1.16.5 version) and so I believe it will be using a 1.16.5 version of multiverse portals. I've tried asking around but no solution has been found, is there anything that would fix it?

I feel like this relates to my issue as well: See #230
It would be nice if someone could try to resolve this issue :)

It would be nice if someone could try to resolve this issue :)
Indeed it would

#230 Does seem related to this only in 1.16.5 so to hazard a guess it's been there for a while.

I just tested this, and I can push hostile mobs through nether portals in 1.21

Closing as no new info and cannot reproduce.


Fix minor issues in linux packaging

Fixed the “low quality application” issue by adding a license file
Included a post-install script to symlink the lighthouse binary to /usr/bin (fixes #131)
Added a description, all packages should have it

Note: It is counterintuitive, but the “-title” parameter of javapackager actually corresponds to the “description” field in dpkg.

Right, so the issue is that UpdateFX expects to find the old and new JARs in that directory so it can calculate a delta between them. Currently all the JARs for each version are in that directory (for me), so blasting it would mean updates would break.
Is there a way to do this without erasing the contents of updates/builds/processed?

Yes, it's easy. We can use -srcfiles to specify a subset of the files in the processed directory to be packaged, in this case it will be $build, postinst, prerm and LICENSE. Then we don't have to erase anything. All versions can stay there, and only the current JAR will be packaged.

Cool! Sounds like a plan.

Maybe it is even easier like this. The plugin is not necessary when we specify individual files. What do you think?

I've squashed and merged this into the lighthouse branch, thanks! Let's get i18n in, and I want to do some more update system tweaks to make the app do a forced/blocking update on first run. Then we can push v29 and I'll rebuild the downloadable packages at that time.


feat: persist settings to local instance (not in browser's local storage)

Describe the solution you'd like
I expected that once I define my settings (such as default model, system prompt, advanced parameters, title generation prompt, etc.), they apply to my user profile. But if I login to my user profile/instance from another device, they are gone and I have the set them anew—they are stored in the browser's local storage (settings key). I'm not sure if this is intended, but having already a user profile, I thought it's more intuitive and practical to save those settings on the instance.
Describe alternatives you've considered
If current behavior is intended, I could imagine having a "settings sync" button, which:

(over)writes current settings in/to the profile/instance (but shows a warning message/popup before doing so);
pulls those settings from the instance if no (local) settings are made yet (which the user can enforce by clearing site data with the browser).

Additional context
Possibly related: #619

You can utilise modelfiles feature to save the preset on the backend!

Thanks, Timothy! Yes, I use model files to specify some model parameters for specific models.
But I believe that model files do not fully address this issue:

they do not store the default model, system prompt, title generation prompt, etc.;
if one prefers a general set of model parameters (temperature, top_p, etc.), one would need to create a model file for each model - possible, but tedious.

I just witnessed (accidentally) another reason for persisting the settings to the local instance: settings are lost when the browser clears the data - loosing all made settings (yes, one could set an exception for localhost - or the analog when accessing remotely - in the browser, but many users may not know this).

Closing in favour of #1022


[CLI] Generated project doesn't run or build

System Information (please complete the following information):

CLI Version: 16.12.2+86688637e3ba4b906f5adf37b767fee124189da7
OS: Windows 11 & Ubuntu 20.04
.NET SDK: 6.0.101

Describe the bug
The console app project generated fails to build and run because:

targets .NET SDK 3.1
is missing package references
is missing using statements.
BuildPipeline method uninplemented.

To Reproduce
Steps to reproduce the behavior:

Train a classification model
In generated project, run dotnet build or dotnet run.

Expected behavior
Generated project builds and runs successfully.
Additional context
Run output

BuildPipeline implementation

Generated project: SampleClassification.zip
Potential steps to fix

Add Microsoft.ML package reference to .csproj file
Set target framework to .NET SDK 6.0
Add using Microsoft.ML; to .training.cs file. Maybe even Microsoft.ML.Data.
Implement BuildPipeline method.

@LittleLittleCloud FYI

The bug is not repro on ML .Net CLI: 16.13.5 and Main branch: <IP_ADDRESS>6101.

Thanks @v-Hailishi for checking. Closing this issue for now.


Remove the DataAvailabilityHeader from the Block

It was discussed synchronously that we don't actually need to have the data availability header in the block. I think this makes sense, as the data root is already in the header, and tying the DAH to the block forces the implementation to include it  in all of the storage/handling of the Block struct.
I could definitely be missing something, could we actually remove the data availability header from the block?

It could be not part of the block in the same way the last commit doesn't need to be part of the block, if you define "block" as just the transactions and messages https://github.com/celestiaorg/celestia-specs/blob/ec98170398dfc6394423ee79b00b71038879e211/src/specs/data_structures.md#block.
There's no downside to decoupling other than engineering time.

It could be not part of the block in the same way the last commit doesn't need to be part of the block

The last commit can't be re-computed by info that is in the block. The DAHeader can though.