Add more metadata to rustc_fingerprint

Previously, rustc_fingerprint was only using the path and mtime for each executable, but this is not always sufficient. For example, Fedora will clamp mtimes to help reproducible builds, so Rust 1.82 looks like /usr/bin/rustc and 2024-10-17 00:00:00 across all builds in Fedora 39 through 42 (rawhide), even though they are different in their full version strings, LLVM, etc.
If the target directory (including .rustc_info.json) is shared across such systems, the fingerprint wouldn't notice the difference, and cargo would try to use artifacts that aren't actually compatible, like:
error[E0514]: found crate `autocfg` compiled by an incompatible version of rustc
 --> build.rs:2:14
  |
2 |     let ac = autocfg::new();
  |              ^^^^^^^
  |
  = note: the following crate versions were found:
          crate `autocfg` compiled by rustc 1.82.0 (f6e511eec 2024-10-15) (Fedora 1.82.0-1.fc42): [...]/target/debug/deps/libautocfg-589c41db1eea6297.rlib
  = help: please recompile that crate using this compiler (rustc 1.82.0 (f6e511eec 2024-10-15) (Fedora 1.82.0-1.fc40)) (consider running `cargo clean` first)

We can improve this situation by adding the file length, although that could also happen to be the same, and the creation date that will match when the file was installed, though not all filesystems support that. All of this comes from a single metadata call, so it shouldn't have any noticeable slowdown that would hurt the caching effort.

It be more robust if we hash the file contents, or even just its .note.gnu.build-id if present (not in rustup's builds), but I suspect that may be more overhead than desired in this function.

Thanks for doing this!
Interesting timing as we recently merged unstable support for switching from mtimes to checksums for determining when to rebuild, see https://doc.rust-lang.org/nightly/cargo/reference/unstable.html#checksum-freshness
Performance does seem to be an issue in that case, so I can understand being cautious here as well.  This is also not as "active" as a case for changes as source.   This seems fine as an incremental improvement that doesn't restrict us on what we do in the future.
@bors r+

:pushpin: Commit b5acf4ce4763ff90fd8bd92494490184a37fa24e has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit b5acf4ce4763ff90fd8bd92494490184a37fa24e with merge<PHONE_NUMBER>a7a673a330a5dd068b7aaa579a265e...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing<PHONE_NUMBER>a7a673a330a5dd068b7aaa579a265e to master...


KeyDBStore refactor so that it just directly takes the DB arguments.

Rather than create an SQL DB, then create a gorm BD using the SQL
DB.  Also split the Create/Get test into two tests.
Signed-off-by: Ying Li<EMAIL_ADDRESS>This also fixes #336

This makes sense. LGTM.


Fixes the imports so it gets the correct files on React NAtive and React Native Web.

This issue was because the react-native line in package.json was pointing to the "web" file so it would give errors.
The fix to to have a single index.js which both platforms use and an implementation.js and implementation.web.js which allows the platform to pick the correct one.
Test Plan
Tests should pass.
@NoemiRozpara or @michalchudziak: Can you check this works correctly for web?
Fixes #30
Fixes #31

@michalchudziak I assume most users will be using this with standard React Native, so I would rather publish this with that as the "default" and Web being the secondary implementation. I think that holds less "risk" but also makes the code clearer as the plain .js file is not full of "not implemented" errors.

@michalchudziak I've just re-read the arguments in the original PR about Webpack needing additional configuration to enable the .web.js file support. In that case, you're right that it should be that way around. Updating the PR now.

An alternative way to get this working is using js/index.native.js or js/index for the "react-native" field in package.json. Looks like specifying the file extension bypasses the .native extension resolution.

@janicduplessis Would that lead to React Native Web correctly picking up the index.js and React Native picking up index.native.js? Have you tested this as I don't use RNW and don't really know the details of how Metro and Webpack handle these cases?

I assume webpack doesn't use the "react-native" field in package.json but I haven't tested.

@matt-oakes Do you want to merge this as is for now just so we unbreak the latest version on npm?

@janicduplessis Sure. It seems like this was confirmed to work. If you have a better solution then please send over a PR 👍


Parse Transformer returning a tuple rather than dict

CubeTransformer().transform(parse_tree) is returning a singleton tuple.
Expecting a dict.
cc: Issue found by @DavidOry

Issue because it will return a tuple if it finds the program type line, and a plain dictionary if not.  Can be fixed by checking type first. Standby.
TRANSIT_LINE_FILE_GRAMMAR = r"""
?start             : program_type_line? lines


Add map wrapping gravity

Currently the gravity only checks distances on a finite square plane surface.
I would like the gravity to also check for distances wrapping through the other side of the plane.

Implemented successfully


test swagger file

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Hi @1openwindow is this PR ready for review?

/azp run automation - sdk

Hi @ArcturusZhang, not ready yet.

Well, please let me know when this is ready for review.

Can one of the admins verify this patch?

Did not see any updates on this PR, closing. Please feel free to reopen or create a new PR when this is ready.


🍒 Cherry pick PR #6252 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/6252 to staging 🍒

🚀 Cherry-picked to staging by @OSBotify in version: 1.1.14-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by @Jag96 in version: 1.1.14-4 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


updated the exit screen #33

Fixed change in #33

When I went through the interview marriage_or_civil_union_VT and the other variables both_cant_divorce_current_state and have_property_agreement were not being used. I am not sure if they're meant to be used but I made it so that if the user enters state that is not Vermont as shown in the picture, the exit screen is triggered.

Resolved on main


fix #15 use simplified api

@Turbo87 will wait for your feedback before merging.

@Turbo87 last chance for input! (for this release)


Document how to enable the phased builds experiment

SUMMARY
We recently enabled phased builds in our monorepo, following the approach in the Rushstack monorepo.
I've documented the various configuration changes to make in a new (experimental) page under the Maintainer section.

Thanks for writing these docs!

The lexicon (script, build, operation, phase, etc) is important for this page. Let me think about it a bit and follow up.

The lexicon (script, build, operation, phase, etc) is important for this page. Let me think about it a bit and follow up.

The thing I'd suggest (maybe you're already on the same page) is that this page should not use those terms wrongly, but, it shouldn't necessarily be the page defining them either.
Maybe there should be a glossary explaining what "The _phase:build phase runs the _phase:build operation, which is defined by the _phase:build script" means 😆


[BUG]: Error setting up entry Anna for plugwise - type conversion

Describe the bug.
When starting Home Assistant, I get a type error in the Anna plugin log. The integration is not set up. For exact error message, see the error log below.
I have been using the integration for a long time. In the 2022.3 release, it is not working anymore.
Expected behavior
A working system.
What version of Home Assistant Core has the issue?
2022.3
What type of installation are you running?
Home Assistant Supervised
How and which version of the integration do you have installed?
Home Assistant Core
What kind of Plugwise device is having issues?
Smile: Anna
What firmware version is your Plugwise product at?
4.2.1?
Home Assistant Logging (optional)
No response
Diagnostics information
No response
Additional information
Logfile of Anna integration:
Logger: homeassistant.config_entries
Source: components/plugwise/gateway.py:53
First occurred: March 8, 2022, 10:35:28 PM (1 occurrences)
Last logged: March 8, 2022, 10:35:28 PM
Error setting up entry Anna for plugwise
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 335, in async_setup
result = await component.async_setup_entry(hass, self)
File "/usr/src/homeassistant/homeassistant/components/plugwise/init.py", line 13, in async_setup_entry
return await async_setup_entry_gw(hass, entry)
File "/usr/src/homeassistant/homeassistant/components/plugwise/gateway.py", line 53, in async_setup_entry_gw
api.get_all_devices()
File "/usr/local/lib/python3.9/site-packages/plugwise/smile.py", line 106, in get_all_devices
self._all_device_data()
File "/usr/local/lib/python3.9/site-packages/plugwise/smile.py", line 60, in _all_device_data
data = self._get_device_data(device_id)
File "/usr/local/lib/python3.9/site-packages/plugwise/smile.py", line 201, in _get_device_data
device_data = self._get_appliance_data(dev_id)
File "/usr/local/lib/python3.9/site-packages/plugwise/helper.py", line 897, in _get_appliance_data
and self._outdoor_temp > data["cooling_activation_outdoor_temperature"]
TypeError: '>' not supported between instances of 'NoneType' and 'float'

@bart1970 let's continue the detailed discussion w.r.t. your problem here.

Hi Bouwe,
Here are the files. The cooling is turned on (with dipswitch).
modules.txt
appliances.txt
direct_objects.txt
domain_objects.txt
locations.txt
Small remark related to the instructions anonymisation. The appliances, domain_objects and direct_objects files also contained the IP address in one of the <measurement lines. Updated the IP address there also to <IP_ADDRESS>.

Thanks! I've downloaded them, you can remove them from the above message.

Bart, a question, how did you end up with Elga as a device-name? Did you rename the OpenTherm device?

You mean the device name in HA I assume. I'm not aware that I have changed this. I don't think I can find that back, can I?

Yes, device name in HA. The thing is, we expect to see OpenTherm as device-name, not Elga.
A quick update, we have found why you see the error, now we need to think about how to solve it...

Good you found it. Hope you can solve it. If I need to do any tests, please let me know.
Just checked some backups. 1.5 year ago, it was already called Elga. So if I changed it, it was already a long time ago.

Yeah OK, makes sense. Due to the HA unique_id, then the name / sensorname stay the same. But I realize now that this does not cause the bug.
We have it working in our test-environment. And we will test in plugwise-beta.
You could help us testing by installing plugwise-beta as a custom_component. Would that be an option for you?
Plugwise-beta can replace Core plugwise 1-to-1, you would only need to install plugwise-beta and restart HA. Plugwise-beta should pick up the existing Plugwise integration so you would not need to do any additional configuration.

@bart1970 @bouwew pw-beta https://github.com/plugwise/plugwise-beta/releases/tag/v0.21.3a1 is available for testing from https://github.com/plugwise/plugwise-beta/pull/248

@bart1970 are you able to test the updates for us?
There are 2 options:

Install plugwise-beta v0.21.3a2
Change the plugwise version in the HA Core files to 0.16.7

We're happy to provide help for both options, if needed.

Was working until an hour ago. I will give it a try now and let you know if I managed.

What will you try?

I copied manually the files (using the file editor) in the custom folder and restarted HA.
Message I get now in the logs look like the old one, but just a little different:
022-03-11 20:41:45 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Anna for plugwise
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 335, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/plugwise/init.py", line 16, in async_setup_entry
return await async_setup_entry_gw(hass, entry)
File "/config/custom_components/plugwise/gateway.py", line 74, in async_setup_entry_gw
api.get_all_devices()
File "/usr/local/lib/python3.9/site-packages/plugwise/smile.py", line 92, in get_all_devices
self._all_device_data()
File "/usr/local/lib/python3.9/site-packages/plugwise/smile.py", line 54, in _all_device_data
data = self._get_device_data(device_id)
File "/usr/local/lib/python3.9/site-packages/plugwise/smile.py", line 226, in _get_device_data
device_data = self._device_data_climate(details, device_data)
File "/usr/local/lib/python3.9/site-packages/plugwise/smile.py", line 172, in _device_data_climate
if not self.cooling_active and self._outdoor_temp > device_data.get(
TypeError: '>' not supported between instances of 'NoneType' and 'float'

Which plugwise-version is shown in the manifest.json file?

"version": "0.21.3a1",

plugwise==0.16.7 <-- that version I mean.

Probably looking at the wrong file. I have in:
/config/custom_components/plugwise/manifest.json
{
"domain": "plugwise",
"name": "Plugwise Beta",
"version": "0.21.3a1",
"documentation": "https://github.com/plugwise/plugwise-beta",
"after_dependencies": ["usb", "zeroconf"],
"requirements": ["https://test-files.pythonhosted.org/packages/97/dd/c03345f77a98bdabd24ff1a8ac60eaf6be3c086126228074f7f66f0ed06e/plugwise-0.16.7a9.tar.gz#plugwise==0.16.7a9"],
"codeowners": ["@CoMPaTech","@bouwew","@brefra"],
"iot_class": "local_polling",
"config_flow": true
}

0.16.7a9, that should work. Why doesn't it work...
Well try some other things.

@bart1970 I think we have solved your issue. Meaning, it works in our testing with the xml-data you provided.
Can you please test with https://github.com/plugwise/plugwise-beta/releases/tag/v0.21.4a1

Today I am not at home. I hope I can test it tomorrow. After testing this, I updated my Ubuntu. That introduced very strange connectivity issues between te containers, but plugwise is in the standard HA container, so think I can test that.
Bouwe Westerdijk @.> schreef op 12 maart 2022 10:24:50 CET:
@. I think we have solved your issue. Meaning, it works in our testing with the xml-data you provided.

Can you please test with https://github.com/plugwise/plugwise-beta/releases/tag/v0.21.4a1
--
Reply to this email directly or view it on GitHub:
https://github.com/plugwise/python-plugwise/issues/158#issuecomment-1065849541
You are receiving this because you were mentioned.
Message ID: @.***>

Thanks! Initialisation works again. Looks fully working.
I do have a question, why are some entities in the configuration tab "sensors" (like elga outdoor temperature) and other entities "diagnostics" (like elga return temperature). Looks like a little artificial difference (maybe due to HA, not due to your integration). The diagnostic information page: https://www.home-assistant.io/integrations/diagnostics/ doesn't help a lot.

Thanks for you feedback!
As for the entities, ran into that myself as well while writing another integration: It's a fairly recent functionality called entity category that does this (see https://developers.home-assistant.io/blog/2021/10/26/config-entity?_highlight=categ#entity-categories). The diagnostics integration 'just dumps' the JSON data, so they are related but not the same thing.

Indeed, thanks, that's good news :)
Also great feedback, is appreciated. We'll consider changing this sensor to the diagnostics category.
We would like to ask you to test the cooling-related operation of the Plugwise-beta integration in detail, when the outdoor temperature is rising and the cooling mode becomes active, if this is possible for you.
We are wondering if our code is working correctly. It is mostly based on assumptions, we really need someone to test the correctness of these assumptions for us :)
If you can, please look at all the details: for instance, are all the climate.anna states shown correct in relation to what the Plugwise App shows? Please note, there is a time delay between what is show in the APP vs what is shown in HA.

About your feedback, we have decided to not change.
Reason: the outdoor_temperature sensor can be available on both on the OpenTherm as the Smile (Adam/Anna) device.
On the OpenTherm device it should be a diagnostics sensor but on the Smile device it shouldn't be one.
Due our implementation it's not possible to separate between the sensors. The only way to fix this is by changing one of the sensor names but this will complicate the implementation in the python code.

The cooling part is at this moment hard to test. For the heating part, some observations.
When the system is heating

The heating icon on the anna HA (Thermostat Card) is off.  The status line on the card is "Heating - Home" which is correct. (See picture below)

The Elga values when ony heatpump:

With gas and heatpump:

If I have other observations, I'll let you know. What is the best way to contact you?

The other observation was already in one of the other tickets. On HA, I can update the temperature (since this release) in steps of 0,1. On the Anna, only in steps of 0,5. Don't think is this is real problem.

On the diagnostics sensor, when I take some time to look at it, it's not so hard :) More later.
On your observations:
The four icons on the climate card indicate the hvac_mode, from left to right:

auto, means a schedule is active, heating or cooling happens based on a schedule
heating, the heating is based on the manually set setpoint
cooling, the heating is based on the manually set setpoint
off, this is artificial mode, a mistake in my view, should not be there

Heating and Home, refer to hvac_action and selected preset.

Why shoud the outdoor_temperature on the OpenTherm device a diagnostics sensor but on the Smile device not.
One is the real outdoor temperature (OpenTherm), the other I aassume based on some internet weather site?

On the pictures, all looks as I would expect.
On the outdoor sensor, on the gateway there is one single sensor, it's common to show it as a plain sensor:

Best way to contact me: just keep posting in this topic. I receive a notice when somebody adds a new message to an existing topic, also when it's closed.

Ah, I think I now understand the icons. Don't know if I find it logical however. If I read it correct, always only one icon is active.
I would expect:

One indication on how you configured your thermostat (for example auto, heating, off, cooling)
In case of auto, the name of the program (for example: Home, Holiday, )
One indication on what the system is really doing: Idle, cooling, heating.
Looks like they are now a little mixed up.

Agree, I share your opinion. That's why I use the Simple Thermostat card instead: https://github.com/nervetattoo/simple-thermostat

I can share my configuration for this card if it helps.

On the control on the HA climate card or the Simple Thermostat card, the Plugwise integration can only turn the schedule off or on. So when you have a schedule active and the cooling mode is off, you could click on the cooling-icon/button and in the end the heating-icon will be shown as active :)

This issue has been solved in v0.21.4 therefore closing this issue.

@bart1970 Can you please test Plugwise v0.23.3a0?
I've made some changes that are specific with Anna connected to a device with cooling capability:

hvacmodes are heat_cool and `auto'
Attributes temperature replaced by target_temp_high and target_temp_low
These two target_temps are displayed in the HA climate card.

Are these changes working in your system?

Hi @bouwew
Yes, but my NUC running Home assistant crashed two weeks ago. Needed to buy one and now make sure everything get working again first... Not much at home, so can take some time.

@bart1970 Thanks for getting back to me, and sorry to hear about the server-problems.
Ok, please get back to me when you are for some testing :)

@bart1970 Plugwise-beta v0.26.0 contains the latest Anna + Elga related updates. According to user huub66 all is working.
When you are up and running again, please let me know if you find anything out-of-order.


(PDB-4161) Update jackson-databind to remediate CVE

To address CVE-2018-5968, update jackson-databind to version 2.9.7, which has
the fix for this CVE (fix was introduced in jackson-databind 2.9.5)

Test FAILed

Test PASSed

CLA signed by all contributors.


PR: Completed Bootstrap Our Services section.

PR: Completed Bootstrap Our Services section.

Looks good.


[MM-43608] - Search limits notification

Summary
Show notification when files or messages are searched and workspace is above limits
Ticket Link
https://mattermost.atlassian.net/browse/MM-43610
https://mattermost.atlassian.net/browse/MM-43608
Screenshots

Admin

Release Note
NONE

/update-branch

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

Thanks @neallred for the review

/update-branch

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/218204

Hey @stevemudie . Gentle ping here

@AGMETEOR I was able to pump the messages up past 10k but I'm not seeing what you show in the screenshots above.
I did however encounter this odd message in the menu that says I'm "close" when I've actually reached 10k.

I'm still working on getting the files past 10 Gig.

@AGMETEOR I was able to pump the messages up past 10k but I'm not seeing what you show in the screenshots above.
I did however encounter this odd message in the menu that says I'm "close" when I've actually reached 10k.
I'm still working on getting the files past 10 Gig.

Hey @stevemudie , they're shown when you're actually above the limits. So we need to go above the limits

Cws@10102020

Shows for messages

@AGMETEOR I was able to pump the messages up past 10k but I'm not seeing what you show in the screenshots above.
I did however encounter this odd message in the menu that says I'm "close" when I've actually reached 10k.
I'm still working on getting the files past 10 Gig.

Hey @stevemudie , they're shown when you're actually above the limits. So we need to go above the limits

Chiming in here: Ditto it needs to go above limits. Other things to consider:

there's a 30 minute lag time in rechecking the usage on the server side.
there's deliberate rounding of the usage number; what's returned from the server is not exact to mitigate information gathering from potential attackers. And the higher the number, the more rounding that happens, up to 1000s of messages.(once 10000 is hit, the next increment isn't shown until 11000 messages).

I'm still working on getting the files past 10 Gig.

@neallred were we able to change the test product to only 1 GB?

/update-branch

Forgot to do that. I have done so now. I ran /update-branch to cause the servers to be restarted so that the cache expires (instance limit cache is long lived at 3 days) but the messages and files that have already been uploaded will be retained.

@AGMETEOR This is all set - sort of.
The banners appear as expected for files and search in the right hand side for both admins and users.
Users see 'Notify Admin' links as expected.
However, clicking the Notify Admin button doesn't appear to do anything as the admin isn't being notified but I expect that's a ticket I'll need to file.
One other issue is when you open up the RHS - without searching - the banner at the top of the files mentions messages.
That too may need a separate ticket. Just let me know.


docs: add Vue example

Q
A

Bug fix?
✖

New feature?
✖

Breaking change?
✖

Deprecations?
✖

Documentation?
✔

Tests added?
✖

Fixed issues
#248

Adding a VueJS example to the docs and Readme.

Nice! Thank you for the quick issue response.
I'll go over it later toady.

LMK if you need more help here, I liked the library!

OK, I just went over your VUE example, so simple and looks great. Thank you!
Just letting you know, I am forking your sandbox and using one with my credentials so that I can maintain it over time. Adding a credit to you, though.

FYI- If you want a nicer message there, just let me know :)

That's fantastic! Thank you.
If we can only add the name of @MaxMonteil it will be great, since he edited my initial draft.
Let us know of more things we can help with 👍

Thanks for the mention!
But I looked at the merged link and I think it still points to the draft before edits. Should I make a new PR with the edited sandbox?
Either way here's the link to the draft with the edits: Sandbox
Similarly let us know if there's more we can do to help 😁

@eyalcohen4 good point. I'll credit @MaxMonteil as well.
@MaxMonteil No need to open a new PR, I'll just update the existing sandbox with your code.


Remove amount parameter from settlements->create()

According to the docs, amount is not a valid parameter.
https://docs.balancedpayments.com/1.1/api/settlements/#create-a-settlement

Thanks, @sophistry.


Password Reset

Uncaught ReferenceError: cannabisdata is not defined
https://console.cannlytics.com/account/password-reset

Thank you immensely for pointing this error out @colburncodes. Long-story short, the password reset functionality has been folded into the website. You can reset your password at:
https://cannlytics.com/account/password-reset
Long story: The various apps have been moving around. The console will become the lims and there is a new app and metrc app as well. The website is now where core functionality will live. This will be resolved in the main branch soon as we merge the app-dev branch shortly.
There was also an issue with HTML element IDs that has been sorted out in the app-dev branch. I will link the code to this issue when it is merged and point out the IDs that were changed.
Thank you for bearing with this and hopefully you can now reset your password and sign into your account.

Thanks @keeganskeate I appreciate the update!


Need to add "related posts" feature

Either "related posts" show up at the bottom of each post, or at least "next" and "previous" buttons.

Disqus is providing this to a limited extent

With next/previous links, tag pages, and Disqus, the readers have enough related posts as it is.


Add S4H upload button

Button to upload PDF report to demonstrate PharMe can be integrated with Smart4Health. Only needs to work on Android.

will not be implemented anymore


query on external storage

hi, i see you closed the issue on saving to external sdcard.  however on my moto g 4g (firstgen) the app (ver3.3) saves to the internal scdard storage and no to my micro sd external card.  the internal storage is at /storage/sdcard0 and the microsd card at /storage/sdcard1,
am i missing something about getting comics to store on the microsd card?
diziet

According to this stackoverflow question (http://stackoverflow.com/questions/5694933/find-an-external-sd-card-location) Environment.getExternalStorageState() doesn't seem to return the actual sd card path on some devices. Would you be ok with an option in the apps settings to set the exact path manually?

that would be cool, would the path be to the root of the microsd card or to the easyxkcd folder if i were to move the app to the sdcard?
diziet

that could work too, feel free to try it out.

The offline path can now be changed since the latest update, try it out!


K-9 mail and letsencrypt

Expected behavior
K-9 connects with an IMAP or SMTP server with a certificate issued by letsencrypt.org without any problems.
Actual behavior
K-9 when trying to connect to an IMAP or SMTP server with a certificate issued by letsencrypt.org returns an invalid certificate warning and asks to accept or decline it.
Steps to reproduce

Add any IMAP or SMTP server with a certificate issued by letsencrypt.org and try to connect.

Environment
K-9 Mail version:
5.010
Android version:
5.1.1 (CM12.1)
Account type (IMAP, POP3, WebDAV/Exchange):
IMAP

Do you have an example smtp server we can connect to (no credentials required, tls happens before auth)? Probably easier this way than searching for one :)

I would expect the set of trust anchors to be a system configuration, not a K-9 configuration.  Do other apps accept letsencrypt-issued certs?  Is the LE CA (or more likely a parent CA) in the system trust anchor list?

It is indeed a system issue if those certs show up as invalid. I've been using LE certs on my own mail server since they entered public beta. My server's hostname is mail.shellvatore.us, and you can test using port 143 for IMAP with STARTTLS.

OK, you are right, this is my fault. Before using LE's certificates I was using self signed ones and the "trust all certificates" option in another client. Then I installed LE's certificates but badly configured the server and did not notice this because of the "trust all" option. Now I have configured everything properly and all problems are gone. I am sorry for bothering and thank you for your work.

One importand detail: one must configure the email server properly with the Letsencrypt certificate AND the CA certificate.
I've found that using stunnel to SSL pop3/imap to popa3d and imapd you must provide stunnel with the full chain cert file (file containing both your certificate and the CA certificate)


Feat/1730 extend filesystem sftp

Description
Extend filesystem source and destination to work with sftp.
As a user, I want to be able to load data from and to sftp server. Probably it can be done already with fsspec

[x] test that it could be done with existing filesystem source
[x] create a credentials type for sftp
[x] add instructions and docs how to do it

A few implementation hints. Look in fsspec_filesystem:

we use last modified / created timestamp for incremental loading. use MTIME_DISPATCH to define a mapping (I hope sftp fsspec has it - upload time should be available)
check FilesystemConfiguration on how we add credentials for particular filesystems (based on protocol)

tests:

we have a fixture (a set of files) that we use for testing on all filesystems
look for where glob_files is tested to start
probably a local sftp server for testing is good
I expect some problems with https certs... how self-signed certs are handled.

Related Issues

Resolves #1730

high quality PR! code and sftp specific tests LGTM. what we need to do is to integrate those tests with other filesystem tests. I think we won't need a separate github ci workflow. we'll also reuse a lot of existing general purpose tests that ie. test the glob funtionality
please try the following:

in tests/.dlt/config.toml add bucket url to the test sftp server
add sftp to ALL_FILESYSTEM_DRIVERS and WITH_GDRIVE_BUCKETS (tests/load/utils)

now all tests that test buckets directly will see sftp.
ie.
@pytest.mark.parametrize("write_disposition", ("replace", "append", "merge"))
@pytest.mark.parametrize("layout", TEST_FILE_LAYOUTS)
def test_successful_load(write_disposition: str, layout: str, with_gdrive_buckets_env: str) -> None:

there are plenty of tests that run filesystem as destination. those are also enabled by the above.
you can skip the whole module (I'm referring to test_filesystem_sftp). see how we skip inactive destinations here:
def skip_if_not_active(destination: str) -> None:

and do the same for filesystems.
when this is done you can merge your test_destination_sftp CI workflow with test_local_destinations.
just enable sftp via
ALL_FILESYSTEM_DRIVERS: "[\"memory\", \"file\"]"

start the sftp server and stuff should run

@rudolfix thanks for the review! setting up the sftp server was fun. I've addressed all the requested changes - could you take another look and let me know if I missed anything?


Ik v1.0

All notebooks rerun on P100. Code updated to latest APIs and cleaned up. Parts of ReadME rewritten to be more concise

Looks awesome! Like the new structure and results


WAR support

Can we expand the file traversal to unpack war files as well?
		if (path.toLowerCase().endsWith(".jar") || path.toLowerCase().endsWith(".war"))

@davemcdonnell
Ah.. good suggestion! I will patch it soon.

@davemcdonnell v1.2.2 released. Would you test this version?

can you add .ear files as well as .war files please.

@cslattery117 Use v1.2.3 release. Thank you for suggestion.


Update gapic-generator hash

to include the following features from gapic-generator:

Remove rogue entrySet call when iterating over maps in C# samples
C# samples: fix incorrect bracing style and and a newline at EOF
SampleGen: import OperationFuture and XxxMetadata for LRO samples

Let's commit it now so we have more CI tests passed, but we'll likely need https://github.com/googleapis/gapic-generator/pull/2979 soon (so one more hash update will be needed).

@alexander-fenster I want to cut a release right away - the last change listed above will unblock us releasing first Java samples :)
I can do another release when the resource name fixes are in.


Add tests for the status package

In the same vein as the main tests.

https://github.com/johanbrandhorst/protobuf/commit/6918a6c46b0349b682ada1b4de2b63424395926a


ci: add preview and publish workflows

Description
This PR includes 2 workflows:

pull_preview which allows maintainers to use /preview command to build and deploy a preview website for PRs.
publish which will automatically publish the website to OSS after push on master branch

Resolved issues
Closes #17
Before submitting the PR, please take the following into consideration

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. If you don't have an issue, please create one.
[x] Prefix your PR title with feat: , fix: , chore: , docs:, or refactor:.
[x] The description should clearly illustrate what problems it solves.
[x] Ensure that the commit messages follow our guidelines.
[x] Resolve merge conflicts (if any).
[x] Make sure that the current branch is upto date with the main branch.

/preview

/preview


Performance issue with observability metrics for unique URLs

Description:
Some of the metrics provided in the observability feature consist of tags such as http_url which is a string with the invocation URL. Therefore, when there are query params or path params of high entropy such as timestamp, the unique metric count rises which may lead to a performance issue.
The following metric is an example. (The URL provided is for the request https://localhost:9095/v2/pet/findByStatus?status=available)
http_response_time_seconds_value{http_method="GET", http_url="/v2/pet/findByStatus_status_available", instance="<IP_ADDRESS>:9000", job="Ballerina_Metrics", protocol="http", resource="get3f443a64f0bf4231b0bbe0435f4a4e73", service="Swagger_space_Petstore__1_0_5"}

Ballerina 1.2.x branch fix is done via https://github.com/ballerina-platform/ballerina-standard-library/issues/1630

As per the discussion [P2] [INTERNAL] (STLSUB-8) - Micro Gateway 3.2.3 goes Out of Memory upon Load... client http.url metric made configurable via https://github.com/ballerina-platform/ballerina-lang/pull/32048

We will need to do the same change for Ballerina Swan Lake as well.

In server side metrics (incoming requests), we can do either of the following

change the value added to the templated path (e.g. /employees/[string employeeId]/details) (prefered)
change the tag to a tracing only tag

In client side metrics (outgoing requests), we can change the tag to a tracing only tag

Note: We can use the following methods to add a tag only to tracing data by using the falling functions

In Ballerina Code: observe: addTagToSpan(tagKey, tagValue)
In native Java code: observerContext.getSpan().addTag(tagKey, tagValue)


Getting Duplicate imports after autocomplete

I haven't narrowed down an exact reproducible case, but I've been getting a lot of duplicate qualified imports on autocomplete lately. At first I thought it related to whether the document was saved or not, but it often works OK (no duplicates) even without saving.

By recently, I mean past month or so.

By duplicate you mean you get the same import line twice?
Given the imports are coming from purs ide, I guess either you're getting bad answers from that or the edit is being screwed up as it is applied
(I'm not seeing this, can anyone else confirm?)

Yes, I mean duplicate qualified imports, like
import Data.Map as Map
import Data.Map as Map

The reason I didn't suspect purs ide specifically is because we've been on a fixed version of it for a while (which is admittedly not the latest), but I'm not aware of any changes to ide import at the purs level.

I'm seeing it too, but not sure how to reproduce. If you want system info:
VSCode version 1.48.2
macOS Catalina version 10.15.6
PureScript IDE version 0.21.4
purs version 0.13.8

purs version 0.13.8

Well, I'm not on the latest then 😆

Example of it happening.

Still not sure what triggers it. What sort of state is maintained in the server that might influence this?

There's no language-server state per se that might be relevant (there is a LS-side model of the document, not our code, I'd be very surprised if that was relevant). Obvious candidates are the IDE server which does the actual import rewriting, or (most likely) the edit that we reflect that back to the editor. I made a change to the edits that we send a little while back as in fact we'd been violating the LSP spec for a long time which was causing problems in other clients. I'd imagine we have some sort of off by one in the edit we're computing.
There's not really enough information there to form an example for me - what were the steps to cause that to happen? ie what did you actually autocomplete, what did you expect to happen to the imports. I'd hazard a guess that you twice completed something on that first line and duplicated the 2nd, ie the last line below the expected edit?
If you could set purescript.trace.server to "verbose" and find the text edit that we send for the imports, that should have the relvant info. Sadly I still can't repro


Upload form not working on geojson-endpoint branch

@willemarcel running into some upload form errors on the geojson-endpoint branch. Seeing bad requests to the https://omk-test.hotosm.org/omk/odk/upload-form endpoint. Is this something we need to configure within the deployment related to the recent url changes?

@smit1678 I succeeded upload a form: https://omk-test.hotosm.org/omk/pages/#/submissions/RH_Shops_and_Markets_V6
Could you send me the files that you tried to upload? Have you tried to upload more than one in a time?

@smit1678 closing this. Reopen if something deserves our attention on this topic


[8.x] Extend api resource wrap and preserveKeys

Currently, wrap only has methods to alter it statically, and preserveKeys can only be altered manually.
The two added methods provide a simple way for altering individual resources.
Example use case for wrapAs():
A collection of user objects with another collection of user objects nested inside it are being returned in a response. The programmer would like to wrap the child users in an attribute called subUsers, but leave the parent users unwrapped.
Example use case for preserveKeys():
Two separate collections of user objects are being returned. The programmer would like one of them to preserve its keys, but not the other.
Both these examples can be handled without the new methods, but not nearly as cleanly. For the preserveKeys example...
$resource = UserResource::collection($userColleciton);
$resource->preserveKeys = true;
return $resource;

becomes
return UserResource::collection($userColleciton)->preserveKeys();

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

@taylorotwell No problems here. I just wanted to bring a post from laravel/ideas to your attention. It's a much more useful and expansive feature that what I've written here, and it fully covers any use cases that my pull request would have handled.
https://github.com/laravel/ideas/issues/1407


[error]lightgbm.basic.LightGBMError: Multiclass objective and metrics don't match

hi,dear all
set the param as follows
param = {'num_leaves': 31, 'objective': 'multiclass','num_class':3,'metrics': ['auc', 'multi_logloss'],
         'learning_rate': 0.08, 'is_unbalance':True,}

what's the matter?
could you please help me ?
thx

Hello @ucasiggcas !
AUC is a binary metric, but your task is multiclass with 3 classes. You can use AUC-mu.
https://lightgbm.readthedocs.io/en/latest/Parameters.html#metric

Hello @ucasiggcas !
AUC is a binary metric, but your task is multiclass with 3 classes. You can use AUC-mu.
https://lightgbm.readthedocs.io/en/latest/Parameters.html#metric

@StrikerRUS
but now has the same bug either
param = {'num_leaves': 31, 'objective': 'multiclass','num_class':3,'metrics': ['auc_mu', 'multi_logloss'],
         'learning_rate': 0.08, 'is_unbalance':True,'max_bin' : 255,'metric_freq' : 1}

what's the problem?
thx

@StrikerRUS
but now has the same bug either
param = {'num_leaves': 31, 'objective': 'multiclass','num_class':3,'metrics': ['auc_mu', 'multi_logloss'],
         'learning_rate': 0.08, 'is_unbalance':True,'max_bin' : 255,'metric_freq' : 1}

what's the problem?
thx

after update the version from 2.3.1 to 3.1.1,
Got Success,
thanks to you and myself
haha

after update the version from 2.3.1 to 3.1.1,
Got Success,
thanks to you and myself
haha

Glad you got it work!
Yes, AUC-mu was released in 3.0.0 version. So the failure on 2.3.1 was expected.
Also please note that is_unbalance

used only in binary and multiclassova applications
https://lightgbm.readthedocs.io/en/latest/Parameters.html#is_unbalance

but not with multiclass objective.

Glad you got it work!
Yes, AUC-mu was released in 3.0.0 version. So the failure on 2.3.1 was expected.
Also please note that is_unbalance

used only in binary and multiclassova applications
https://lightgbm.readthedocs.io/en/latest/Parameters.html#is_unbalance

but not with multiclass objective.

@StrikerRUS
so how to deal with the data imbalance in multiclass?

@StrikerRUS
so how to deal with the data imbalance in multiclass?

You are free to use any third-party solution or class_weight from https://github.com/microsoft/LightGBM/issues/3702#issuecomment-753535634 or sample weights (https://github.com/microsoft/LightGBM/issues/3687#issuecomment-753552824).

You are free to use any third-party solution or class_weight from https://github.com/microsoft/LightGBM/issues/3702#issuecomment-753535634 or sample weights (https://github.com/microsoft/LightGBM/issues/3687#issuecomment-753552824).


Checks should be able to filter subjects who have events launched 7 days ago with missing sheets

[ ] Launched events missing sheets (7 days)

This requires a specified event, and a date offset, and return subjects who have that event with sheets that haven't been entered since the event date, and also return a list of these unentered sheets.

Closing "checks" related issues in favor of future re-imaginations.


[Bug]: AppScope init container can't extract library on distroless images

Steps To Reproduce
When running AppScope on Kubernetes, the scope-pod-extract-0 init container injected by the mutating webhook fails to start when the container image uses a distroless base image, as these images contain no shell.
When the container attempts to run, it uses the following entry point:
/bin/sh -c mkdir /scope/0 && /scope/scope excrete
Which will fail with a similar message:
failed to create containerd task: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: exec: "/bin/sh": stat /bin/sh: no such file or directory: unknown

Environment
- AppScope: 1.4.0

Requested priority
None
Relevant log output
No response

The PR #1516 address the problem.
QA instruction
Checkout to the branch specified in the PR #1516
# build the AppScope
make build CMD="make all"
make image

# start cluster
kind create cluster
kind load docker-image cribl/scope:dev

# deploy the scope k8s configuration
docker run -it cribl/scope:dev scope k8s --metricdest tcp://scope-prom-export:9109 --metricformat prometheus --eventdest tcp://other.host:10070 --debug | kubectl apply -f -
kubectl label namespace default scope=enabled

# deploy examples distroless configuration observe no issue

To deploy example distroless configuration You can do following
git clone https://github.com/GoogleContainerTools/distroless.git
cd distroless/main/examples/go
docker build -t my-image:latest .
kind load docker-image my-image:latest
kubectly appl -f example.yaml

example.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: my-deployment
spec:
  replicas: 1
  selector:
    matchLabels:
      app: my-app
  template:
    metadata:
      labels:
        app: my-app
    spec:
      containers:
      - name: my-container
        image: my-image:latest
        imagePullPolicy: IfNotPresent
        ports:
        - containerPort: 8080

@jrcheli
I've extended #1516 with manual test - it maybe usefull for You to verify the fix.
Side note: If we decide to fix this issue in 1.4.0 - we should modify milestone reference in this issue.


Bitfinex don't submit market order without price (or price = zero)

Don't submit market order without price, or price = zero. When i try, the response is: InvalidOperationException: Nullable object must have a value..
When you use market order type in trading, the price doesn't matter, you just want to open/close a position. If ExchangeSharp submit

Thanks for filing this bug. If you send a pull request with the solution, we'll be happy to merge it. If you don't know how, you can also post the updated code here.

Sure Bro! Working on it :D


could not locate box "svg"

Hello
Thanks for the great product!
So, running standalone octant on a Linux VM works just fine with these env variables set
ENV OCTANT_LISTENER_ADDR=<IP_ADDRESS>:8888
ENV OCTANT_DISABLE_OPEN_BROWSER=true
But, when running it inside the Docker container I get the following error as well as missing whole side navigation bar:
2020-02-11T21:08:47.918Z        ERROR   api/navigation_manager.go:97    load namespaces {"component": "websocket-client", "client-id": "b1fdc8f3-4d12-11ea-a749-001c142f4ffc", "poller-name": "navigation", "poller-instance": "79ab6608-546f-4d9b-8fb1-8f2f8ea7e48c", "err": "generate entries for Workloads: could not locate box \"svg\""}

Just wonder if you know the answer. Thanks much in advance!

It finally worked. Rebuilt the binary using your dockerfile. Thanks.


🐛 [cloud_functions] Flutter Web - "<getObject: NoSuchMethodError: The method '-' was called on null.>" Returned For Callable Function

Bug report
Describe the bug
When calling a callable function on Flutter Web it fails. The function works as expected on Mobile (iOS and Android) but seems to break when using Flutter Web.
Both the assignments to the HttpsCallable and HttpsCallableResult types are very slow. When they finally are completed, they both contain the same error:
<getObject: NoSuchMethodError: The method '-' was called on null.>
In the VS Code sidebar you can see the error gets assigned to the callable variable:

The debugger window prints out the following:
Expected a value of type 'Map<String, dynamic>', but got one of type 'List<dynamic>'
But this is a result of the Map containing the error string not being assignable to the List that is expected. The error already occurs when the HttpsCallable variable is assigned, so that "Expected value" error message is irrelevant.
Steps to reproduce
The code that is being run:
Future<List<MatchupModel>> getMatchups({int count = 10, List<Tuple2<int, int>> excludeMatchups = const []}) async {
    try {
      final HttpsCallable callable = CloudFunctions.instance.getHttpsCallable(
        functionName: 'getScandalsForMatchup'
      );
      final HttpsCallableResult response = await callable.call(<String, dynamic>{
        'userId': await deviceIdService.getDeviceId(),
        'count': count,
        'exclude': excludeMatchups.map((matchup) => '[${matchup.item1},${matchup.item2}]').join(','),
      });
      final List matchups = List.from(response.data as List);
      return matchups.map((matchup) => MatchupModel.beforeVoting(List<Map>.from(matchup as List))).toList();
    } on CloudFunctionsException catch (e) {
      print('caught firebase functions exception');
      print(e.code);
      print(e.message);
      print(e.details);
    } catch (e) {
      print('caught generic exception');
      print(e);
    }
    return [];
  }

Expected behavior
It should work the way it works on mobile - a list of matchups is returned. Instead I just get the error. For now I am forced to make traditional HTTPS calls which are not secure. We chose cloud_functions precisely because it was (supposed) to be implemented well with Flutter and handle authentication under the hood.

Additional context
I've tried importing the javascript library to index.html as noted in #2558 and tried returning my data in a map such as in #2548. Neither have fixed the issue.

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
[√] Flutter (Channel beta, 1.22.0, on Microsoft Windows [Version 10.0.18362.1082], locale en-US)
    • Flutter version 1.22.0 at C:\Users\davem\fvm\versions\beta
    • Framework revision d408d302e2 (7 days ago), 2020-09-29 11:49:17 -0700
    • Engine revision 5babba6c4d
    • Dart version 2.10.0

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at C:\Users\davem\AppData\Local\Android\sdk
    • Platform android-29, build-tools 29.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 3.6)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)

[√] VS Code (version 1.49.3)
    • VS Code at C:\Users\davem\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.15.0

[√] Connected device (4 available)
    • ONEPLUS A6003 (mobile) • 6e45a5a8   • android-arm64  • Android 9 (API 28)
    • Web Server (web)       • web-server • web-javascript • Flutter Tools
    • Chrome (web)           • chrome     • web-javascript • Google Chrome 85.0.4183.121
    • Edge (web)             • edge       • web-javascript • Microsoft Edge 85.0.564.68

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.10.0
Flutter SDK 1.22.0
scandalus 1.0.0+1

dependencies:
- auto_size_text 2.1.0 [flutter]
- cloud_functions 0.6.0+1 [meta flutter firebase_core cloud_functions_platform_interface cloud_functions_web]
- cupertino_icons 1.0.0
- device_info 0.4.2+8 [flutter device_info_platform_interface]
- dio 3.0.10 [http_parser path]
- firebase_core 0.5.0 [firebase_core_platform_interface flutter quiver meta firebase_core_web]
- flip_card 0.4.4 [flutter]
- flutter 0.0.0 [characters collection meta typed_data vector_math sky_engine]
- flutter_search_bar 2.1.0 [flutter meta]
- font_awesome_flutter 8.8.1 [flutter]
- google_fonts 1.1.0 [flutter http path_provider crypto pedantic]
- linkify 2.1.0
- loading_overlay 0.2.1 [flutter]
- localstorage 3.0.2+5 [flutter path_provider]
- overlay_support 1.0.5 [flutter async]
- paulonia_cache_image 0.1.0 [flutter firebase_storage firebase path_provider hive http]
- percent_indicator 2.1.7+4 [flutter]
- pl_grid 0.2.1 [flutter]
- platform_alert_dialog 1.0.0+2 [flutter]
- provider 4.3.2+2 [flutter nested collection]
- reveal_search_bar 0.2.1 [flutter pedantic]
- shimmer_animation 0.1.1+1 [flutter]
- skeleton_text 1.0.0 [flutter]
- sleek_circular_slider 1.1.0 [flutter]
- slide_popup_dialog 0.0.2 [flutter]
- tuple 1.0.3 [quiver]
- url_launcher 5.7.2 [flutter url_launcher_platform_interface url_launcher_web url_launcher_linux url_launcher_macos url_launcher_windows]
- uuid 2.2.2 [crypto convert]

dev dependencies:
- flutter_launcher_icons 0.8.1 [image args yaml path]
- flutter_native_splash 0.1.9 [yaml image color]
- flutter_rename_app 1.0.0 [flutter yaml process_run ansicolor]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher meta source_span stream_channel string_scanner term_glyph typed_data]
- lint 1.3.0

transitive dependencies:
- ansicolor 1.0.5
- archive 2.0.13 [crypto args path]
- args 1.6.0
- async 2.5.0-nullsafety.1 [collection]
- boolean_selector 2.1.0-nullsafety.1 [source_span string_scanner]
- characters 1.1.0-nullsafety.3
- charcode 1.2.0-nullsafety.1
- clock 1.1.0-nullsafety.1
- cloud_functions_platform_interface 2.0.0 [flutter meta firebase_core plugin_platform_interface]
- cloud_functions_web 2.0.0 [cloud_functions_platform_interface flutter flutter_web_plugins firebase http_parser meta]
- collection 1.15.0-nullsafety.3
- color 2.1.1
- convert 2.1.1 [charcode typed_data]
- crypto 2.1.5 [collection convert typed_data]
- device_info_platform_interface 1.0.1 [flutter meta plugin_platform_interface]
- fake_async 1.2.0-nullsafety.1 [clock collection]
- ffi 0.1.3
- file 5.2.1 [intl meta path]
- firebase 7.3.0 [http http_parser js]
- firebase_core_platform_interface 2.0.0 [flutter meta plugin_platform_interface quiver]
- firebase_core_web 0.2.0 [firebase firebase_core_platform_interface flutter flutter_web_plugins meta js]
- firebase_storage 4.0.0 [flutter firebase_core]
- flutter_web_plugins 0.0.0 [flutter characters collection meta typed_data vector_math]
- hive 1.4.4 [meta crypto]
- http 0.12.2 [http_parser path pedantic]
- http_parser 3.1.4 [charcode collection source_span string_scanner typed_data]
- image 2.1.18 [archive xml meta]
- intl 0.16.1 [path]
- io 0.3.4 [charcode meta path string_scanner]
- js 0.6.2
- matcher 0.12.10-nullsafety.1 [stack_trace]
- meta 1.3.0-nullsafety.3
- nested 0.0.4 [flutter]
- path 1.8.0-nullsafety.1
- path_provider 1.6.18 [flutter path_provider_platform_interface path_provider_macos path_provider_linux path_provider_windows]
- path_provider_linux 0.0.1+2 [path xdg_directories path_provider_platform_interface flutter]
- path_provider_macos 0.0.4+4 [flutter]
- path_provider_platform_interface 1.0.3 [flutter meta platform plugin_platform_interface]
- path_provider_windows 0.0.4+1 [path_provider_platform_interface meta path flutter ffi win32]
- pedantic 1.9.0
- petitparser 3.0.4 [meta]
- platform 2.2.1
- plugin_platform_interface 1.0.2 [meta]
- process 3.0.13 [file intl meta path platform]
- process_run 0.10.12+3 [path collection io yaml meta args pub_semver]
- pub_semver 1.4.4 [collection]
- quiver 2.1.3 [matcher meta]
- sky_engine 0.0.99
- source_span 1.8.0-nullsafety.2 [charcode collection path term_glyph]
- stack_trace 1.10.0-nullsafety.1 [path]
- stream_channel 2.1.0-nullsafety.1 [async]
- string_scanner 1.1.0-nullsafety.1 [charcode source_span]
- term_glyph 1.2.0-nullsafety.1
- test_api 0.2.19-nullsafety.2 [async boolean_selector collection meta path source_span stack_trace stream_channel string_scanner term_glyph matcher]
- typed_data 1.3.0-nullsafety.3 [collection]
- url_launcher_linux 0.0.1+1 [flutter]
- url_launcher_macos 0.0.1+8 [flutter]
- url_launcher_platform_interface 1.0.8 [flutter meta plugin_platform_interface]
- url_launcher_web 0.1.4+1 [url_launcher_platform_interface flutter flutter_web_plugins meta]
- url_launcher_windows 0.0.1+1 [flutter]
- vector_math 2.1.0-nullsafety.3
- win32 1.7.3 [ffi]
- xdg_directories 0.1.0 [path process flutter]
- xml 4.5.1 [collection convert meta petitparser]
- yaml 2.2.1 [charcode collection string_scanner source_span]

Apparently this error is an issue with the VS Code variables section in general...even if I set a breakpoint in the very first line of code, before any of the Firebase code is called, I see that error.
Anyone else experienced something similar?

@davidmednikov
Since this issue isn't related to Firebase and seems the original issue has been fixed per your comment here, I am closing this.


future 0.18.3

Recipe directory diff between current master and this update:
diff --git a/recipe/meta.yaml b/recipe/meta.yaml
index 1820f44..90dfeee 100644
--- a/recipe/meta.yaml
+++ b/recipe/meta.yaml
@@ -1,4 +1,4 @@
-{% set version = "0.18.2" %}
+{% set version = "0.18.3" %}

 package:
@@ -7,7 +7,7 @@ package:

 source:
   url: https://pypi.io/packages/source/f/future/future-{{ version }}.tar.gz
-  sha256: b1bead90b70cf6ec3f0710ae53a525360fa360d306a86583adc6bf83a4db537d
+  sha256: 34a17436ed1e96697a86f9de3d15a3b0be01d8bc8de9c1dffd59fb8234ed5307

 build:
   number: 1

Jira ticket:  (-)
The upstream data:
Github releases:  https://github.com/PythonCharmers/python-future/releases
Diff between the latest and previous upstream releases
License: https://github.com/PythonCharmers/python-future/blob/master/LICENSE.txt
Requirements:

setup.cfg:  https://github.com/PythonCharmers/python-future/blob/v0.18.3/setup.cfg
setup.py:  https://github.com/PythonCharmers/python-future/blob/v0.18.3/setup.py

Actions:

Notes:
*
Additional info:

Package's statistics

Priority C | effort: easy | Category: anaconda | subcategory:  | pkg_type: python
Outdated platfroms: 'linux-64', 'linux-aarch64', 'linux-s390x', 'linux-ppc64le', 'osx-64', 'osx-arm64', 'win-64'
Latest version: 0.18.3
Release on PyPi:

version: 0.18.3
date: 2023-01-13T03:15:17

PyPi history
Popularity:

3 months downloads: 856486.0
All time:  10214552 downloads -  future  0.18.3  Clean single-source support for Python 3 and 2

Other checks:

[x] https://github.com/PythonCharmers/python-future/tree/v0.18.3 exists

[ ] The upstream url error:
http://python-future.org

The home url must have HTTPS!
3. - [ ] Check the pinnings
4. - [ ] Changelog url error:
https://github.com/PythonCharmers/python-future/tree/v0.18.3
200
5. - [ ] Additional research
https://github.com/PythonCharmers/python-future/issues

[ ] dev_url error:
https://github.com/PythonCharmers/python-future

[ ] doc_url error:
http://python-future.org

The doc_url must have HTTPS!
8. - [ ] Verify that the build_number is correct
9. - [ ] Verify if the package needs setuptools

[ ] Verify if the package needs wheel

[ ] NO pip in test

[ ] Verify the test section

[ ] Verify if the package is architecture specific

[ ] Verify that private modules are not mentioned in the recipe For example: (_private_module)

[x] license_file: LICENSE.txt is present

[x] license_family MIT is present

[x] License: MIT
[x] License is spdx compliant

Check if the license identifier has correct name from the SPDX License List

Identifier

MIT

MIT-0

MIT-advertising

MIT-CMU

MIT-enna

MIT-feh

MIT-Modern-Variant

MIT-open-group

MITNFA

Check dependency issues:

../aggregate/future-feedstock/recipe/meta.yaml
Dependencies: ['pip', 'python']
noarch:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

linux-64:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

linux-ppc64le:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

linux-s390x:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

osx-64:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

osx-arm64:

py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

win-64:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

Package Build Score: 18

Links:

Anaconda Recipes feedstock
Update branch
conda-forge recipe
PyPI
Diff between upstream and feature branch
Diff between origin and feature branch
The last merged PRs

Updating the recipe:
If the recipe needs additional modification the update branch can be modified. Note that the PR diffs are not updated with these changes.
git clone -b 0.18.3 git@github.com:AnacondaRecipes/future-feedstock.git

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_58h5odgjnt/clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_58h5odgjnt/clone/recipe/meta.yaml:30: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_58h5odgjnt/clone/recipe/meta.yaml:45: http_url: http://python-future.org is not https
Errors were found


Fixing links to API references

What does this PR do?
The links to API references that are auto-generated do not have the correct base URL. This PR manually adds links to the API references.
Example of autogenerated link: https://huggingface.co/docs/optimum.graphcore/main/en/add_support_for_new_model#optimum.graphcore.modeling_utils.PipelineMixin.parallelize
Actual (correct) link: https://huggingface.co/docs/optimum/graphcore/add_support_for_new_model#optimum.graphcore.modeling_utils.PipelineMixin.parallelize

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


(Restricted) data are not cleared on logout

Admin logs in on browser, logs out, Joe logs in on the same browser, has the cache of Admin.
Joe won't receive any updates he is not allowed to receive, but can still access the (cached) data.
The only thing stopping him from doing so is that the data is not shown. Still, it is a security concern, and I have had cases where I could see data I was not allowed to (user names).
sessionStorage.clear(); on logout might resolve this; we will need to go fetching all the resolves Joe should have access to when Joe logs in, which might affect the user experience in terms of speed.

This will affect #2295, as with this, the "beginning" will not be on page load, but on user login.

I can not reproduce this with current master.


Let TIs and applicants work on applications together

We've heard that TIs (CBOs) like to train applicants to apply for programs on their own. They want to split the work, so applicants can slowly start picking up more and more of the application, until they're self-sufficient. Younger applicants, in particular, tend to be keen to learn to apply by themselves.
So let applicants and TIs work together, either in real-time or asynchronously. Share the state of an application across the two of them.

I would like to assign myself to this issue. Thank you!


Change: Pyblish Wait for pool futures to finish

Changelog Description
Wait for publish pool to finish before closing pyblish.
Testing notes:

Regular publish

When testing on my machine with the sync queue open next to my publish, I still had blank remote lines when publish finished. I don't know if it's just the time the UI updates itself, in which case it should be fine.


Failed to resolve: org.larq:lce-lite:0.4.0

Describe the bug
After doing a Gradle sync on the Tflite Android Image Classification example application, Android Studio throws this warning and the app cannot be built.
To Reproduce
Follow the steps here, namely:

Add the below to the app-level build.gradle:

allprojects {
    repositories {
        maven {
            url  "https://dl.bintray.com/plumeraihq/larq-compute-engine"
        }
    }
}

Add the below to the lib_support build.gradle:

dependencies {
    ...
    implementation 'org.larq:lce-lite:0.4.0'
    ...
}

Expected behavior
Should be able to sync with gradle without any issues.
Environment
Windows 10, Gradle 4.0.0, Android Studio 4.1.2,
TensorFlow version: NIL
Larq version: 0.4.0
I suspect this is caused by how the nested maven metadata file does not declare the latest version as 0.4.0 because this problem goes away when I replace implementation 'org.larq:lce-lite:0.4.0' with implementation 'org.larq:lce-lite:0.1.2' instead

No worries, thanks for this awesome library too!
I don't have the maven CLI installed but it worked locally on the Tensorflow-lite Image Classification example app with the following changes:

Downloaded the latest .aar from the link you placed, and placed it in ./android\lib_support\libs
Do the following to lib_support/build.gradle:

Remove:
    implementation fileTree(dir: 'libs', include: ['*.jar'])
Replace with:
    implementation fileTree(dir: 'libs', include: ['*.jar', '*.aar'])

Remove:
    implementation 'org.tensorflow:tensorflow-lite:0.0.0-nightly-SNAPSHOT'

Remove:
    implementation 'org.tensorflow:tensorflow-lite-support:0.1.0'
Replace with:
    implementation ('org.tensorflow:tensorflow-lite-support:0.1.0') {
        exclude module: 'tensorflow-lite'
    }

Thank you very much for figuring that out. If it's ok with you, then we'll copy this information into our docs for future users. (Then we can remove the maven-related things from the docs).

Yeah for sure, feel free to do so!

On 9 Feb 2021, at 11:13 PM, Tom Bannink<EMAIL_ADDRESS>wrote:

Thank you very much for figuring that out. If it's ok with you, then we'll copy this information into our docs for future users. (Then we can remove the maven-related things from the docs).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi, I am also trying to go through the turorial here about builing an Android App. And I tried the method of @tehtea, but I guess it does not work for me though.
about the bug
I succeed to build on Android Studio, but fail to run, and it reports like this at the running stage:
E/tensorflow: ClassifierActivity: Failed to create classifier.
    java.lang.IllegalArgumentException: Failed to load XNNPACK delegate from current runtime. Have you added the necessary dependencies?

E/tensorflow: CameraActivity: Exception!
    java.lang.NullPointerException: Attempt to invoke virtual method 'void android.graphics.Bitmap.setPixels(int[], int, int, int, int, int, int)' on a null object reference

reproduce
And here's what I did all the way:

download the examples from here.
Go to Build > Select Build Variant and select  supportDebug for TFLite_Image_Classification_Demo_App.app and otheres for debug to use the solution of lib_support instead of lib_task_api.
Download the aar file here, and excute the command
to install it in my local maven.

mvn install:install-file \
    -Dfile=lce-lite-v0.5.0.aar \
    -DgroupId=org.larq \
    -DartifactId=lce-lite -Dversion=0.1.000 -Dpackaging=aar

add mavenLocal() in the build.gradle file under Android folder, and it looks like this.

allprojects {
    repositories {
        mavenLocal()
        google()
        jcenter()
    }
}

Follow the steps mentioned by @tehtea to modify build.gradle under lib_supoort folder , However, I use localMaven for the lce.
and it finally looks like this

dependencies {
    implementation fileTree(dir: 'libs', include: ['*.jar'])
    implementation project(":models")
    implementation 'androidx.appcompat:appcompat:1.1.0'
    implementation 'org.larq:lce-lite:0.1.000'

    // Build off of nightly TensorFlow Lite
    //implementation 'org.tensorflow:tensorflow-lite:0.0.0-nightly-SNAPSHOT'
    implementation 'org.tensorflow:tensorflow-lite-gpu:0.0.0-nightly-SNAPSHOT'

    //implementation 'org.tensorflow:tensorflow-lite-support:0.1.0'
    implementation ('org.tensorflow:tensorflow-lite-support:0.1.0') {
        exclude module: 'tensorflow-lite'
    }
    // Use local TensorFlow library
    // implementation 'org.tensorflow:tensorflow-lite-local:0.0.0'
}

Build and run.

PS: The app may quite unexpectedly when running on real device.
I'm new to Android, so if there's anything that written ambiguous, feel free to reply!
Any help would be appreciated!
Thanks!

We have just updated our documentation thanks to your input, see here for details.
I think all the issues reported here are now resolved, so I'm closing this. Feel free to re-open if you think otherwise - or open a new issue if you have encountered any other problems with LCE on Android.


Add description to the examples

It would be great if the examples contains a description of the example. In earlier versions of Popper, most examples was quite clear (to me), but some of the newer iggp-* examples are not that clear.
The examples are surely described a little more in some of the Popper papers, but a (perhaps short) description would clarify what problem the example is solving.

I've started to (finally) do this. Will keep it in mind in the future, thanks!


[Discussion] Env reset seems to be much slower than previous versions

I timed env.reset() method on my M1 chip MacBook Pro, and it took almost 10 seconds, that is much slower than before. I tracked down deep into the code calling stack and found it was caused by the line in BaseEngine.warmup():line672:
Pedestrian.init_pedestrian_model()

I stopped digging deeper and just comment out that line, and everything works fine as usual since I don't need any pedestrians 😂
I don't know the exact reason for this or it just happens on macOS and won't be slower in other platforms.
Hope it will be fixed in future 🙏 🌹

Thanks for letting me know. 🌹
This preload process is for avoiding the jitter when a human model is created for the first time. It does take some time to load the model, so I preload it to memory when the simulation is initialized. This preload is enabled by default. You can turn off this by setting preload_models=False in env_config. Actually, it doesn't take too much time on Windows and Linux. But given your feedback, it would be better to disable the preload function or determine when to do the preload more wisely.

Thanks for the suggestion, it works🙏!


Media Session API integration

Integrate media session api into our player. This allows us to set the media meta & controls for the browser media controls.

Nice!


dependency graph re-evaluation issue

see example:
http://create.vizor.io/uucidl/draw_scaled_image/edit
In this sketch I have a texture fed into a graph that will render it to the framebuffer
After you load the sketch for the first time in your browser, and you press play, nothing is shown.
Press edit into the draw_image subgraph. You will see the texture resolution is read as 0x0
Now press stop and play again, the texture resolution is now 512x512 and the picture shows.

I'm not sure this is happening anymore, I haven't seen the effect of this lately. Closing for now, but if anyone sees this with particular steps, please feel free to reopen.


checkov: 2.3.259 -> 2.3.261

Diff: https://github.com/bridgecrewio/checkov/compare/refs/tags/2.3.259...2.3.261
Changelog: https://github.com/bridgecrewio/checkov/releases/tag/2.3.261
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

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

2 packages built:

checkov
checkov.dist


Create A Webpage For This Repository

The only way to increase the reach for this repository is via creating a webpage (possibly hosted on Github Pages) with all the programs in all the programming languages. This way it becomes easy to share the content of this repository to every person who could benefit from the content of this repository.
The Tech Stack completely depends on you whatever you are comfortable in projecting this repository in form a static webpage.
I would really appreciate people working on this issue. The Repository has complete patterns with all code files for the languages - C#, JAVA, PYTHON. What's left is some programs in C++ and Complete C Pattern programs.

Sound good.
How do you want the webpage to be like?
I can try...

Sound good.
How do you want the webpage to be like?
I can try...

Hey there buddy, thanks for reaching out!, So basically I want a website in a way that it populates my github repo with all its content- should be intuitive since majorly students will be using it. Feel free to come up with any queries regarding the repo, I'm pretty active here!
Tech stack-depends on you
UI- Depends on you and I can give my inputs for the same
Feel free to use any frameworks if you want, just make sure it is host-able on Github Pages!

Hey there buddy, thanks for reaching out!, So basically I want a website in a way that it populates my github repo with all its content- should be intuitive since majorly students will be using it. Feel free to come up with any queries regarding the repo, I'm pretty active here!
Tech stack-depends on you
UI- Depends on you and I can give my inputs for the same
Feel free to use any frameworks if you want, just make sure it is host-able on Github Pages!

You can browse through the repo and understand the aim behind its creation!
A user should be able to find the code from the huge pattern collection that exists in the repository.
Feel free to connect for better communication!
LinkedIn:https://www.linkedin.com/in/arya--shah/
Twitter:https://twitter.com/aryashah2k

You can browse through the repo and understand the aim behind its creation!
A user should be able to find the code from the huge pattern collection that exists in the repository.
Feel free to connect for better communication!
LinkedIn:https://www.linkedin.com/in/arya--shah/
Twitter:https://twitter.com/aryashah2k

You can check this website I designed - bitphoton.ml
If you like it I can design this in the same way,
P.S: I am NOT a pro, just an amatuer.
P.P.S: I am also a java dev and I see that all java tasks are taken. Feel free to assign (if required) to me in the future of such tasks, I shall more than happy to help.
Cheers!!

You can check this website I designed - bitphoton.ml
If you like it I can design this in the same way,
P.S: I am NOT a pro, just an amatuer.
P.P.S: I am also a java dev and I see that all java tasks are taken. Feel free to assign (if required) to me in the future of such tasks, I shall more than happy to help.
Cheers!!

Sure buddy, the Java section has two sections still left and you can definitely join the existing contributors there, I'll assign you those issues, you can check them here #30 #28 If you're comfortable in working on them, then do go ahead.
Also for the website thing, I checked out your work and its great and might suit my repository's purpose, you could create a prototype website for my repo and I can check if we can proceed with that further!
Sounds Great?

Sure buddy, the Java section has two sections still left and you can definitely join the existing contributors there, I'll assign you those issues, you can check them here #30 #28 If you're comfortable in working on them, then do go ahead.
Also for the website thing, I checked out your work and its great and might suit my repository's purpose, you could create a prototype website for my repo and I can check if we can proceed with that further!
Sounds Great?

Yes, I am delighted you liked it.
I shall start the work on the prototype right away.
Question: Does the project have a official logo, symbol anything ?? or does it have to be a just the name?

Yes, I am delighted you liked it.
I shall start the work on the prototype right away.
Question: Does the project have a official logo, symbol anything ?? or does it have to be a just the name?

Well Well, I havent thought on a project logo, might ask my designer friends to get me one made, is there any way you can hold that and create without need for that? In turn, let your creativity also reflect! You can go ahead and add a logo from your side or simply add a dummy one for now to suit the purpose!😊

Well Well, I havent thought on a project logo, might ask my designer friends to get me one made, is there any way you can hold that and create without need for that? In turn, let your creativity also reflect! You can go ahead and add a logo from your side or simply add a dummy one for now to suit the purpose!😊

I have an initial preview of the website. You can check it out here
Known improvement scopes/issues:

The page isn`t responsive i.e it does not scale well on mobile widths and heights
The links do not work, I will add those ones after other pages (about, and browse) are done.

You might want to change the color theme of the website. Currently it black and white.
You might also want to change the fonts used. I open to those inputs.
Hope you like it.
ChlorinePentoxide

I have an initial preview of the website. You can check it out here
Known improvement scopes/issues:

The page isn`t responsive i.e it does not scale well on mobile widths and heights
The links do not work, I will add those ones after other pages (about, and browse) are done.

You might want to change the color theme of the website. Currently it black and white.
You might also want to change the fonts used. I open to those inputs.
Hope you like it.
ChlorinePentoxide

also it does deploy on Github Pages, I am just using Netlify here since I had an account there and it works faster.

also it does deploy on Github Pages, I am just using Netlify here since I had an account there and it works faster.

Thats wonderful, I saw the website and it suits the purpose and is pretty well for version 1.0! (I hope you will be working with me in the future too😊)
Some Inputs:

Please could you change the theme to a dark background?
Something like this:

Align this section also to the left only because most of our users would be accessing it via phone or tablet, so trying to make their troubles less!

Do notify me when the buttons are linked and I can actually browse through the website in shoes of a user.

PS: YOU HAVE DONE A GREAT JOB!!!! You can go ahead with the enhancement of the website and make your very valuable contribution.

Thats wonderful, I saw the website and it suits the purpose and is pretty well for version 1.0! (I hope you will be working with me in the future too😊)
Some Inputs:

Please could you change the theme to a dark background?
Something like this:

Align this section also to the left only because most of our users would be accessing it via phone or tablet, so trying to make their troubles less!

Do notify me when the buttons are linked and I can actually browse through the website in shoes of a user.

PS: YOU HAVE DONE A GREAT JOB!!!! You can go ahead with the enhancement of the website and make your very valuable contribution.

I so very glad you liked it. I do something up my sleeve to make it responsive.
I will align it to the left.
Sure, I will notify you when the links work.
Cheers,
ChlorinePentoxide

I so very glad you liked it. I do something up my sleeve to make it responsive.
I will align it to the left.
Sure, I will notify you when the links work.
Cheers,
ChlorinePentoxide

Hey there,
Are the patterns same across all language or is it specific?
If they are the same could you mind linking them here so I can use them for the website?
ChlorinePentoxide

Hey there, the patterns are mostly same across all languages, I'm adding the links to the readmes for each language here, hope that was exactly what you were trying to convey!
C -https://github.com/aryashah2k/Printing-Pattern-Programs/blob/main/C Pattern Programs/README.md
C++-https://github.com/aryashah2k/Printing-Pattern-Programs/blob/main/C%2B%2B Pattern Programs/README.md
C#-https://github.com/aryashah2k/Printing-Pattern-Programs/blob/main/C%23 Pattern Programs/README.md
Java-https://github.com/aryashah2k/Printing-Pattern-Programs/blob/main/Java Pattern Programs/README.md
Python-https://github.com/aryashah2k/Printing-Pattern-Programs/blob/main/Python Pattern Programs/README.md

If they are the same, then that will save me a lot of work.
Thanks
P.S Currently creating the Browse pages, drop some suggestions if required

Cool , I ran down all the folders and concluded that all the patterns are same for all the languages, each language has the same number of patterns so you can have images from any one folder and do it for the rest of the languages!

Indeed I would love to discuss the layout of the webpage, what is the progress and if you could send a screenshot and I could suggest some ideas!

Checkout this
I have not started work on the responsiveness and dark themes, though I will be implementing it very soon, after the browse
pages and code pages have been done.
Question: How do we want the code to be displayed, in the sense should we redirect the user to a different webpage (which is easier and looks best) or appear like a pop up message (it is definitely more user friendly but will be nightmare for syntax highlighting and website responsiveness because the source isn`t going to be small)?
Cheers,
ChlorinePentoxide

Hey there, I checked it out, it's perfect and exactly aligns the way I pictured it, So I believe that the way you described-"in the sense should we redirect the user to a different webpage (which is easier and looks best)" is the best way to take it forward!
Buddy trust me I've grown fond of the work you've done and really looking forward to the final product!
You might have better understanding with the designing of the webpage and feel free to implement your way, I'll also make suggestions here and there, but it is just my view point and as a friend you can always correct me and make me learn from your point of view!

Also what I've understood behind how the webpage will work is that the patterns will be displayed and from the top left

So, as we change the language, the links for the code behind the patterns will be changed for that language right?

When the user selects a specific pattern, the javascript in the website processes the pattern code and the selected language to redirect the user to a specific language implementation of the pattern. This approach could not have been possible if the patterns were not same across the different language. So really condensed my work.
I am not sure if it would look exactly good, but the headings which says "Alphabetic Patterns" needs to go, and we need color code the pattern types or just leave it numerically increasing so the internal links still differentiate the pattern types but not to the website user (The reason is the way the patterns are displayed. The current implementation allows it look exactly same and remain usable on smaller displays but do not allow for headings to be displayed without compromising the responsiveness).
Also, the current design I have thought of takes the source code directly from the repository. The advantage is subsequent changes is directly reflected. The minor changes have to be done for this to work. Some web browsers do not like non-alphanumeric characters like "#,$.." in the directory paths. So that might have to change for this to work. For instance the root directory for C# (C# Patterns) has the character '#' in the path.
Hopefully you will do the needful,
ChlorinePentoxide

Got it, so for changes on my part, I need to get rid of # in the C# so do I make it CSharp? Will that work?

Yes that will do. Check other paths too.
Please also do tell whether we go color coded or increasing numbers.
The preview for color-coding is available here as well.

So, the color coded ones is perfect and you can proceed with that, I'm checking all the paths and getting rid of all the signs that may cause trouble.

Perfect. I will move on and other patterns.
I am currently enjoying this thoroughly.
Thanks for the opportunity,
I will let you know when the first complete working demo is possible, which is very soon.
Cheers,
ChlorinePentoxide

Pleasure working with you buddy! I've made the changes you requested and we are good to go!
I respect the fact that you considered taking source code directly from the repository since the repo is still not complete with codes for a few languages, so that really takes care of the state of website which gets dynamically updated as the repository progresses towards completion.

Yes its best to change that too.

A small correction required, the images in subfolders series and wave patterns do not fit the standard(all others) size of 160x160 pixels you may want to fit the patterns to that size for uniformity in display space. Do note that it is still possible to work with non-standard image size but will stand out because it will look different.
Another small discrepancy I noticed is that all the images JPEG formated which is fine except for Pattern 49 in pyramid patterns which is saved as PNG.
You can check the new color schemes here

Hi, so I shall proceed making those wave pattern images into 160x160 size?
I'll also make sure to convert that png image to jpeg
The new color scheme is awesome! Would you be doing further improvements into making the complete background dark? Let me know, The work till now is amazing!!!

Lets make everything into standard size of 160x160.
Yes I will be working on the dark color scheme once the code pages have been done.
Thanks for the response.
Cheers
ChlorinePentoxide

I've made all the changes requested! You can proceed further without any hindrance...!

I've made all the changes required:
The images are now uniform size 160x160
The Pattern 49 is now jpeg
Thanks a lot for taking your time out on a sunday to work on this, really appreciate it!

No problem. I usually feel bored on weekends.
Check the website out now. It has added search functionality.
Known Issues: I do know the search isn`t working perfectly and will look to fix it very soon.
I will be moving on to the code pages. If you have any inputs please do share.
ChlorinePentoxide

That's wonderful, the website is really improved and feels populated now, also you can move forward wityh adding for the remaining two pattern types which are now in 160X160 resized(Series and Wave Patterns)
The fikter pattern does work only thing is to find a workaround to sort of bring the filtered content right to the top. I tried with the lowest situated pattern and had to scroll right below in order to see my filtered patterns. Usually the website users might not take that much effort and end up thinking the functionality is not working properly, You can try your best to get that sorted, if it doesn't happen then we might just add a note stating to scroll down to view your filtered content...
For code pages, I trust you on this the way you feel right to be done, as we discussed yesterday, you can move forward with that...(linking the repository code directly to the code page)

I have preview for the code pages here
Shall I go ahead and generate the html files for all the source files?
Few points to note:

The source uses a webservice called EmGithub which takes a Github file and embeds it into a webpage using a static link of the file.
I see some code has not be written (for C and C++), the EmGithub links can be generated but will show an error on the page if viewed before its development.
The filenames must not be changed after the creation of links for EmGithub. So whatever filenames are used to generate will be stuck if it is not updated in the future.
Therefore I suggest before we go ahead the source files can be made into standard naming conventions such as snake case (for python) and pascal case (for java). Most source files have been named without taking into account the conventions.

Upsides:

The code will automatically be updated if the file in the master branch is updated.
There are different themes EmGithub can operate in so dark theme implementation is going to be easier.

Cheers
ChlorinePentoxide

Hi, I checked the source page and it's neat and well and suits the purpose,
Regarding the case of missing source code files for c and c++ what do you suggest should be done?
Do I add empty files in .c and .CPP formats for the missing patterns as of now and you link them and layer any contributor or myself could update it with the code inside and will the changes reflect in the emgithub generated?

I suggest you add an issue regarding naming conventions for different languages, so I can generate the code based on those patterns, any voluntary contributor will know where and how exactly he/she must frame the code, and emgithub will operate without any hiccups in the foreseeable future.
Please cover the naming conventions, how to name alternative method files, alignment of code and depth which comments for code must cover.
For example here is a sample convention for this repository:

The source code for a specific pattern must have the same name as the pattern
If there are two method for the same pattern, then both the methods shall lie within the same source.
The name of the file shall carry the same name as the pattern and it must reside in the same directory as the pattern type (for eg. source for Alphabetic Pattern 1 must reside in the directory "alphabetic patterns" and must be named Pattern1.java or pattern1.py/c/cpp/cs)
No two inner class name declaration must be the same for a specific pattern type
The documentation/comments/variable names/method names must clearly reflect its purpose
The code must be properly aligned and clearly readable
The following contributors must preserve the effort of the existing contributions by not removing the name of the previous contributors in the source files

This is a just a suggestion. Feel free to modify to suit the amount of restrictiveness of the code.
Hopefully you shall find this useful.
P.S: I might only be able to work on weekends from now. Hopefully you would have done the changes by then.
Cheers
ChlorinePentoxide

Sure I'll add an issue regarding naming conventions for pattern code files.
As of now, every contributor has quite maintained the naming convention without it being explicitly stated and mostly I will only be the one who shall take up the remaining two languages code files so rest assured, the convention shall remain the same...
Also for alternative patterns, I'm going to inform individual contributors to simply merge the alternative code in a single file so no issues will exist for cases for alternative patterns.
I just wanted to ask that of I add blank .c and .cpp files alreadu witut he correct naming convention followed till now will the source page get updated once code is updated in that blank file?
Also thanks a lot for all your efforts, you've been a valuable addition to this small family of contributors to this repository. Just like you there was this one guy who singlehandedly took care of the while c# section of repository, I'm so glad that open source has given me this chance to interact such great people like you...sure you can handle stuff on your weekends...I'm glad you took so much of efforts towards this

Just let me know about this:
I just wanted to ask that of I add blank .c and .cpp files alreadu witut he correct naming convention followed till now will the source page get updated once code is updated in that blank file?

Don`t mind. I will take care of it. :)

Hey @ChlorinePentoxide , I have merged all alternative programs into a single file for various patterns in all the language folders. I hope all the hurdles have been cleared... Do let me know how the webpage is progressing, I havent added Blank Files for pending pattern programs , let me know if you want me to do that... Apologies to ping you on a weekday, you can acknowledge this message during the weekend, take your own pace ... grateful for all the efforts so far!

Thanks!! Saves me a lot of work.

Yup no issues, take your own time, there's no hurry! As and when you can take time out for this project, it's up to you how to proceed...!

So sorry, I didnt update you on the progress this Sunday. I havent updated the netlify cloud website. I will be doing that this Sunday. Here is the progress I have made so far:

Added extra themes
Made it responsive, finally (Subject to testing, lol)

With regards,
ChlorinePentoxide

Hey @ChlorinePentoxide , thanks for the update, waiting for this Sunday to view your creation, by then the Java section shall also be compete hopefully, I have personally requested the contributors to help speed up the completion of pending sections, I myself am personally working on C and C++ Section to ensure timely completion!!

Hey there,
We hit a roadblock...
The service I had intended to use (EmGithub) for dynamic sourcing, is not working properly.
I am afraid that we have to fallback.
Instead I intend to create an application that will generate the required web pages from the sources automatically. The only downside being this has to be done manually.
Now onto some good news,
The web design part is almost over. I intend to open a pull request tomorrow that will add all the website functionality.
With regards,
ChlorinePentoxide

Hey there, no issues, since adding manually will be a lot of hard work, I encourage you take your time and maybe figure out an alternative for emgithub. Also thanks for completing work on design part, cant wait to see how the webpage looks !! Eagerly waiting for the PR.
Take care,
Regards,
Arya

I had already searched alternatives for EmGithub, and the short answer is there are none.
The good news is the development for the application is almost over and it works like magic.
The application is responsible for doing all the tasks like creating the webpage for individual pages.
Here is shortlist of all the things it is able to do currently:

Construct web pages from templates
Take information from config files for automated link management
Use rules file for syntax highlighting in code
Generate CSS headers for multiple themes using style files
The application is built from scratch and tailor made for this repository and use case only.
I have attached the screenshot of its working below.

I could not finish it by today, sorry. But it is great progress so far.
P.S: I had to reorganize the project structure i.e that is the basic layout of the project. Hope you are fine with it.
P.P.S: Just a suggestion. Would you consider opening a Gitter chat room for this project. Commenting on this issue will only make it grow bigger. :)
Till next time,
With regards,
ChlorinePentoxide

I accidently pressed unassign, sorry,
you can reassign me

Hi there, that's great progress, no issues, take your time...I shall open a gitter chat room for this project by today. Havent used gitter before if there are any suggestions to that then do let me know!!

Please do check the gitter, often !!
:)


Add WatchInstances to Service interface

@xichen2020 @robskillington @martin-mao @mmihic

Coverage remained the same at 89.897% when pulling 7b39f925266c32f2a998d541a46242d0e03b0f87 on chao-watch into d5bfbf4742812704b67d01085fafb8a49e68556e on master.

Coverage remained the same at 89.897% when pulling e9978e2558d28cdf51141134ea20e66815cb5568 on chao-watch into d5bfbf4742812704b67d01085fafb8a49e68556e on master.

Coverage remained the same at 89.897% when pulling 1b1fd674e5d9c9e06debe2443200d945e5abdc4f on chao-watch into d5bfbf4742812704b67d01085fafb8a49e68556e on master.

Coverage remained the same at 89.897% when pulling 1be4f5527cedf2af71e8b55004ba174a8b0bae1a on chao-watch into d5bfbf4742812704b67d01085fafb8a49e68556e on master.

LGTM


Improve diagnostic messages for imported helpers

This PR improves the diagnostic messages for imported helpers, changes how imported helpers are checked, and removes NodeFlags enum members related to emit helpers that are no longer needed.
// cc: @vladima, @mhegazy

Looks good otherwise, just need merge conflicts addressed


auto sync

i think this plugin need a feature to sync all styles with other browser in chrome (all browsers logged in with google account) and other browsers
thank you

Yes I agree. I had wrongly assumed that Chrome would sync the settings if I were logged. Please enable support for this.

closing as duplicate of #230