Tech Controlers integration not loaded

Since I upgraded to Home Assistant 2021.6.4 the Tech Controllers integration is not loaded. Even with a clean Home Assistant install (version 2021.6.4) the integration is not loaded.
The error message that appears: Setup failed for tech: Integration not found.
Is this a know issue?
Greetings,
Martijn

It looks like missing version information in manifest.json

Owner should do it, my suggestion is:
{
  "domain": "tech",
  "name": "Tech Controllers",
  "config_flow": true,
  "documentation": "https://www.home-assistant.io/integrations/tech",
  "version": "0.0.1",
  "requirements": ["tech"],
  "dependencies": [],
  "codeowners": [
<EMAIL_ADDRESS>  ]
}

I did add a extra line to the manifest file
—> “version”: “1.0”,

in the clean installation of HomeAssistant 2021.6.4, the error has disappeared, there is only a orange announcement in the log about not tested with HomeAssistant, and possible …blablablabla.
The only thing problem is that the “tech” is not in the add-on list to add it.

when I take a old version of HomeAssistant like 2021.3.2, I can install the “tech” add-on. When I upgrade the HomeAssistant to 2021.6.4 and add the “version” line in the manifest file, it is working, with only the orange message in the log about not tested etc.

so, for me it is working now, just upgraded to 6.5. It only could be difficult for new users or people who start from a fresh 2021.6.4 version to install the tech add-on.
thanks for the help. I will close tomorrow this case

Yes, I have absolutely the same problem. But unfortunately I have removed the integration, and now I can't add it again. (Also I have added version in manifest file, but it doesn't help. So, the problem is actual.  I also had removed it, but I could set a backup back of a older version.  I hope they can fix the intergration soon. Do you see something in the home-assistant.log file?

 I also had removed it, but I could set a backup back of a older version. I hope they can fix the intergration soon. Do you see something in the home-assistant.log file?

Mmm, nothing interesting. Only this:
2021-06-17 09:39:34 WARNING (SyncWorker_3) [homeassistant.loader] We found a custom integration tech which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
But there is some magic here: today morning I have tryed to find this integration one more time, and this time everything is ok! So, my problem solved.

Oh good one, …. Maybe some fairy 🧚♀ did come.

Thank you guys for sharing this. I will update main branch once I find some free time.
Cheers, Mariusz

looks like it's not a simple missing version line from the manifest.json.
After i've put the version line i get the next error in the logs:
Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/helpers/integration_platform.py", line 33, in _process     platform = integration.get_platform(platform_name)   File "/usr/src/homeassistant/homeassistant/loader.py", line 498, in get_platform     cache[full_name] = self._import_platform(platform_name)   File "/usr/src/homeassistant/homeassistant/loader.py", line 503, in _import_platform     return importlib.import_module(f"{self.pkg_path}.{platform_name}")   File "/usr/local/lib/python3.8/importlib/__init__.py", line 127, in import_module     return _bootstrap._gcd_import(name[level:], package, level)   File "<frozen importlib._bootstrap>", line 1014, in _gcd_import   File "<frozen importlib._bootstrap>", line 991, in _find_and_load   File "<frozen importlib._bootstrap>", line 973, in _find_and_load_unlocked ModuleNotFoundError: No module named 'custom_components.tech-controllers.media_source' 2021-06-17 16:14:08 ERROR (MainThread) [homeassistant.helpers.integration_platform] Unexpected error importing tech/logbook.py Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/helpers/integration_platform.py", line 33, in _process     platform = integration.get_platform(platform_name)   File "/usr/src/homeassistant/homeassistant/loader.py", line 498, in get_platform     cache[full_name] = self._import_platform(platform_name)   File "/usr/src/homeassistant/homeassistant/loader.py", line 503, in _import_platform     return importlib.import_module(f"{self.pkg_path}.{platform_name}")   File "/usr/local/lib/python3.8/importlib/__init__.py", line 127, in import_module     return _bootstrap._gcd_import(name[level:], package, level)   File "<frozen importlib._bootstrap>", line 1014, in _gcd_import   File "<frozen importlib._bootstrap>", line 991, in _find_and_load   File "<frozen importlib._bootstrap>", line 973, in _find_and_load_unlocked ModuleNotFoundError: No module named 'custom_components.tech-controllers.logbook' 2021-06-17 16:14:08 ERROR (MainThread) [homeassistant.helpers.integration_platform] Unexpected error importing tech/group.py Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/helpers/integration_platform.py", line 33, in _process     platform = integration.get_platform(platform_name)   File "/usr/src/homeassistant/homeassistant/loader.py", line 498, in get_platform     cache[full_name] = self._import_platform(platform_name)   File "/usr/src/homeassistant/homeassistant/loader.py", line 503, in _import_platform     return importlib.import_module(f"{self.pkg_path}.{platform_name}")   File "/usr/local/lib/python3.8/importlib/__init__.py", line 127, in import_module     return _bootstrap._gcd_import(name[level:], package, level)   File "<frozen importlib._bootstrap>", line 1014, in _gcd_import   File "<frozen importlib._bootstrap>", line 991, in _find_and_load   File "<frozen importlib._bootstrap>", line 973, in _find_and_load_unlocked ModuleNotFoundError: No module named 'custom_components.tech-controllers.group'
I'm still trying to understand what happening - it is looking for files which are not exists (yet) - logbook.py.

Did you place all te files in the directory 'custom_components.tech-controllers’ ?
I thought all files needed to be placed in the directory ‘custom_components.tech’

no, i’ve edited only the json file, with those additional 2 lines(version
and cloud connection). everything else remains the same.
On Thu, 17 Jun 2021 at 17:02, Tijn1974 @.***> wrote:

Did you place all te files in the directory
'custom_components.tech-controllers’ ?
I thought all files needed to be placed in the directory
‘custom_components.tech’
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mariusz-ostoja-swierczynski/tech-controllers/issues/20#issuecomment-863265912,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOQQY7C7LPQWRNE4Q4QMRU3TTH57LANCNFSM46Y77TPA
.

Try to rename the directory name, if you look in the Json file, there is also written “tech” as directory and on the wiki page on this site it’s the same.

This is how it works fine for me:
{
"domain": "tech",
"name": "Tech Controllers",
"config_flow": true,
"documentation": "https://www.home-assistant.io/integrations/tech",
"version": "0.0.1",
"requirements": ["tech"],
"dependencies": [],
"codeowners": [
<EMAIL_ADDRESS>],
"iot_class": "cloud_polling"
}

Thank you guys for sharing this. I will update main branch once I find some free time.
Cheers, Mariusz

Hi,
When it will be?

@jontofront  This is finally done in dev branch. Please test it out and report any issues. I will marge is to master hopefully soon.
Few words of explanations it took so long. During the development of this feature I was trying to get some simple support from Tech (answer on their APIs), but they didn't provide it although they promised it. After long time of reminding myself to them, winter was over and I lost both time and motivation. This winter I found out how many people were interested in this integration and even some people put effort for further development of this integration and so I regained bit of motivation. Anyway I don't have anything against forking this repo and moving forward if I lost it again :)


Add HVAC / Rear Window Heat temperature controls

I am thinking of starting to implement this feature. Since there are some unknown variables here, I am offering to make a first draft and try to iterate through this together.
What we know?

Mapping from here (considered draft status): https://github.com/orgs/SAIC-iSmart-API/discussions/11
Probably different Vehicle models have different mappings
Model can be determined from ASN v1.1 from series or maybe modelName (if someone can help interpret what these bytes mean)
Handle remoteTemperature as a retained message that "survives" gateway restarts, and is always used when sending commands to the car.

Using the topics from here:

https://github.com/orgs/SAIC-iSmart-API/discussions/6

Since this will already be startet, try to see if backWindowHeat can be implemented (though I am not sure if it will be testable before winter)...
My proposal is to make an implementation trying to consider these aspects, and ask people with different car configurations to test it and improve it.

So, first draft is out there, @tisoft, feel free to test it out, I will be retesting this week.
For other models, I am not aware of any different mappings, but I set the first stone on how it could look like in the future. It would also be possible to allow users to generate a yaml config that can configure the whole mapping, however I am not sure if we want end-users to potentially send invalid data to the SAIC API... So this part should be further discussed...

The different App versions contain several mappings. It is unclear to me which of them are really used.
India:
https://github.com/SAIC-iSmart-API/decompiled-app/blob/9932f224a5179d82352a06df4801319018b00d24/sources/com/saic/cmp/business/home_page/air_conditioner/View/AirConditionerAdapter.java#L48-L302
Europe:
https://github.com/SAIC-iSmart-API/decompiled-app/blob/0ef0061c2c9a64dd75afe6afdea6f428f0d768e8/sources/com/saic/cmp/business/home_page/air_conditioner/View/AirConditionerAdapter.java#L51-L215
Australia:
https://github.com/SAIC-iSmart-API/decompiled-app/blob/c0e4776c43ca3464a3afba1e9d1e530fe359bac0/sources/com/saic/cmp/business/home_page/air_conditioner/View/AirConditionerAdapter.java#L48-L165
Thailand:
Seems to be unsupported?

OK, nice to see, here some parts I found in the Europe Version as reference:
   public static final String DEFAULT_EH32_VEHICLE_NAME = "MG4 Electric";     public static final String DEFAULT_EP21_VEHICLE_NAME = "Marvel R Electric";     public static final String DEFAULT_EP22_VEHICLE_NAME = "MG5 Electric";
This is called this way:
          if (isEH32()) {                 getClimateControlDelegate().airConditioning(getTemperatureByteEH32(this.temperature), this.temperature);             } else {                 getClimateControlDelegate().airConditioning(getTemperatureByteEP21(this.temperature), this.temperature);             }
Other methods like getTemperatureByte and getTemperatureByteZS11E are dead code, so it is really only handling the MG4 differently.
About the regions, I am assuming from what I saw, that the only difference is the temperature range, but I will check it further. However this mapping seems like the way to go, maybe plus region variations... Good that all the info is in the vinlist...

One more thing I will be trying to cover:

Resend ACC Command to car if AC is on "front" or "on" mode, and temperature is changed, so the car gets updated temperature setting.

The base change has been merged, resending AC commands on temperature change is not yet implemented


distributed runners

As far as I can see, strider's runners are built into strider main process. What do you think about distributed runner, that might run on the same host, different host, or even on your local machine, like Gitlab CI's.
BTW, I love the fact that strider has an (option for) built-in runner.

I managed to do it, by overriding the simple runner spawn config. and making it ssh into the slave machine. the problem is that I didn't get it to work on multiple slaves, yet.

@kayoub5 check out Strider-CD/core, it's the latest work on a parallelized/distributed setup.

Any updates for this feature ? I have 11 projects in the strider and dedicated server with 8 cores, but strider is build only one project at the same time. So server are use only one core. Any way to run different projects build parallel ?

See my comment above, and feel free to comment there. I don't see any work being done of the current version to move this forward, unless someone has a good amount of time to dedicate to making the runners multi-process based. I don't have that time.

Ok, Strider-CD/core and Strider-CD/drone - it is a new version of strider ? I can try to realize this feature. Some docs about strider architecture very helpful to me, how it work in general

Basically jobs come into strider, and strider sends them to the runner which runs the plugin code based on you configuration. Check out https://github.com/Strider-CD/strider-runner-core and the simple and docker runners in this org.


[Task] Update logos that make OC possible

What projects is this story for?
OpenChallenges
As a user, I want
NA
Description

Remove AWS logo as we probably aren't authorized to use it
(Optional) remove the blue border around the ITCR logo
Use Sage logo with its name as the logo/icon is not known enough to recognize the org

Acceptance criteria
No response
Tasks
No response
Anything else?
No response
Have you linked this story to a GitHub Project?

[X] I have linked this story to a GitHub Project and set its metadata.

Duplicate of #1829


webhook

I'm creating a project in which I need the server to send me the result of the match to a webhook, is it possible?

Hi @macshinaider,
Is the webhook triggered after server shutdown or after each match?
I think the options here are:

You create your own downstream fork of my container image which includes your webhook script and you modify entry.sh in your image to execute it.
I add support to my image for users to:

Supply a bundle of customisations, e.g. as a tar gzip fetched from a URL; and
Override the call to cs2 to call a script of their choosing instead. This script could:

Execute custom steps before launching cs2 (pre-hooks)
Execute a wrapper that watches the STDOUT of cs2 and triggers custom actions on certain output strings, like match ending.
Execute custom steps after cs2 closes (post-hooks)

I don't know when I will have time to work on option 2, so option 1 may be your best bet if you are in a hurry.

TKS 😍


fix: comment the explanations inside pr template

Description
Noticed that sometimes contributors forget to remove the explanations from their Pull Request.
Also, it's easier for reviewers and maintainers to detect empty parts because they will really be empty!
Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[x] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Risk
No risk
Testing
Can't really test until it's merged

I understand why you've suggested this change, but I'm not convinced it's the right approach.
Commenting out the placeholder instructions makes them useless, as notes such as the below won't be seen by contributors, and we'll likely get lower quality PR descriptions as a result:
If applicable, please link to the github issue and put `closes #XXXX` in your comment to auto-close the issue that your PR fixes.

I understand why you've suggested this change, but I'm not convinced it's the right approach.
Commenting out the placeholder instructions makes them useless, as notes such as the below won't be seen by contributors, and we'll likely get lower quality PR descriptions as a result:

If applicable, please link to the github issue and put `closes #XXXX` in your comment to auto-close the issue that your PR fixes.

I won't cry if you don't accept it, feel free to merge or close it depending on what the team choose

I would prefer to get an empty PR description than a description full of explanations, it's way easier to detect.
Comments are existing in order to be read, overwise we wouldn't comment our code

I don't really get why contributor wouldn't see it as a comment, as it's almost the same when you create the Pr, the only difference is when you already created it


Safari: settingRowFocus context key not reset after switching away from it

VSCode Version:
OS Version:

Steps to Reproduce:

Open Web/Codespace in Safari
Open a text file
Open Settings UI editor
Switch back to the text file
Press enter, nothing happens

[2020-10-16 10:55:18.212] [window] [info] [KeybindingService]: | Resolving [Enter]
[2020-10-16 10:55:18.215] [window] [info] [KeybindingService]: \ From 23 keybinding entries, matched settings.action.focusSettingControl, when: settingRowFocus, source: built-in.

The keybindings troubleshooting log indicates the settingRowFocus context key is not set to false.

Does this issue occur when all extensions are disabled?: Yes/No

No longer reproducible on latest OSS.


docs: add notation in pull-request template

Since not a few people submit PRs by providing only a link with their own usage only, I decided to add a notation to the PR template like below

TODO: Update this comment and add some reference links.
your usage of the word: link of github or something
The documentation for the word: link for word documentation

[!NOTE]
To get approval, you need to show external documentation proving that the word is not your own creation, but is common in the public and industry.
Please check CONTRIBUTING.md for detail.

@mitsudome-r @h-ohta
Could you review this?
To merge this pull-request, I need approval from either of you.

I checked operation in https://github.com/tier4/autoware-spell-check-dict/pull/840


RuntimeError: Error deserializing object of type int

Looks like we need to download this file: http://dlib.net/files/shape_predictor_68_face_landmarks.dat.bz2
and put it in the trained model folder

With Git LFS, if you download the project as a "zip" instead of cloning it, you'll miss this file. You can read: https://github.com/antoinelame/GazeTracking/issues/9


Add leofang as author

DO NOT MERGE yet, need to talk to a few internal folks first...

Another question: Do we plan to acknowledge fundings? Is this common in SciPy papers?

Re: funding. Good question. @rgommers you have thoughts here? IIRC, if a paper originates from private (non-grant) based funding (e.g., engineers from a company publish a paper on company time), the funding is not typically explicitly mentioned.

No need for funding mentions I'd say, we captured that in blog posts and talks already, and there are no requirements to include it (as there would be for grants). Grant-funded scientific work is also typically fully-funded, while we have a mix of companies putting in hard dollars, and folks who contributed either as part of their day job or outside of that. So it's nontrivial to give accurate credit in 1-2 sentences.

Thanks, Ralf/Athan. Updated my PR.
btw, I don't know the criteria for :equal-contributor: and I'll leave it up to you to decide. I noticed Stephan didn't add it (#5).

Just want to confirm this is OK to merge now.
I guess we'll need to figure out how we want to order the authors and how we want to do equal contributors. I guess there's "equal contributor" and also "corresponding". See for example, https://raw.githubusercontent.com/scipy-conference/scipy_proceedings/2018/papers/project_jupyter/paper.rst.

@asmeurer Go ahead and merge and we can sort out order, etc, later.

Yeah, I have no opinion in sorting or equal-contribution criterion 🙂


Phase one of color variable work

:+1: Per our discussion in slack, the goal is to merge this branch into master before the end of the next sprint.


Hold press to download sent pictures.

In other matrix clients like vector.im, you can right click on a sent image and download it. Could we get a feature where you can hold press on a sent picture to download it?

I assume you talk about the vector.im webclient.
It is possible to do the same thing on the android client
1 - tap on the image to display it : its download should start.
2 - When the download is finished, leave the full screen display and tap on the image side (not on the image itself) : a menu should be displayed with a SAVE entry.


Issues with number of sentences, using inverted commas, cannot take user input dynamically.

Great work first of all. I was working with this module when I came across these issues:
We cannot use less than 5 sentences
Cannot take dynamic input from user, so it can only be run by putting texts as input in the code
Size of image cannot be resized, better if we could get a pdf output.
Great project anyways.

Hi, @0LUCIFER , thanks for highlighting the issues. Can you please elaborate on what do you mean by dynamic input. For the image resize and pdf output (good ideas) we'll update you on what can be done.

Great that you liked the ideas. By dynamic input what I meant was that I
tried taking user input using the input() method in python and then feed it
into the pywhatKit text to handwritten function but the image was not
getting generated in a proper way which led to the image not opening.
Here is the code I tried-
import pywhatkit
convert=input("Enter text")
pywhatkit.text_to_handwriting(convert,"destination path and file name", rgb
values in a list)
On Sun, 16 May 2021, 12:33 am aryan, @.***> wrote:

Hi, @0LUCIFER https://github.com/0LUCIFER , thanks for highlighting the
issues. Can you please elaborate on what do you mean by dynamic input. For
the image resize and pdf output (good ideas) we'll update you on what can
be done.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Ankit404butfound/PyWhatKit/issues/64#issuecomment-841709395,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP23J4KQFOSTCHRPTHSLBQLTN3APHANCNFSM446EZVLA
.

We should be able to take input like the above, can you please tell us the exact error you get?

As no further information regarding the issue has been provided so I am closing it.


add multisig to integration tests

It takes just under 3 minutes on my machine. There are probably a lot more things that can be run concurrently to shorten it.

@lenkan Why did only 2 integration tests run with this build?

@pfeairheller This was branched from development. So the tests from #146 are not included. But I can rebase this branch now, then they will run.

This PR is superseeded by #150 if you decide to go with that one.


Fix #12 correct aliases for UMD

Hey @andykog
Thanks for looking into this! It now worked for globals but I noticed it broke when using webpack (which uses commonjs module system), so I extended on the changes: https://github.com/mobxjs/mobx-react-devtools/commit/6a41575f9c4d3f6808b984b9da9a1f6e0d74b29b
Thanks nonetheless.

Ah, you're right, didn't notice that.


fix parallel evaluation of derived optimizables

fix the derived optimizables Residue() and Quasisymmetry() so that  least_squares_mpi_solve can be run in with more than 1 worker per group.
code would call Residue.J(), the workers would skip and return nothing whilst the Leader would execute, and hang on expecting communications from its workers.
Fixed and a rudimentary unit test added.

This change to spec.py looks right.
If I understand correctly, the change in boozer.py does not end up having any effect. Is this your understanding too? Consider any worker process that previously would have hit line 310 (return np.array([])) before hitting 313 (self.boozer.run()). In the changed boozer.py, these worker processes now all call self.boozer.run(), but immediately exit that function on line 119, since the logic in line 117 is the same as what is now line 310. So there is no change in behavior. I'm not opposed to the change in boozer.py, just want to make sure we are on the same page.
Because there is no change to the behavior of boozer.py, the new test test_Quasisymmetry_paralellization passes even with the previous version of boozer.py. It's fine to add this test for the sake of extra coverage. But did you intend for the test to catch something that was failing before this PR?
Would it make sense to combine test_derived_optimizables_mpi.py and test_integrated_vmec_mpi.py into a single file? (Maybe test_integrated_mpi.py - it should have "mpi" in the filename so the CI knows to run it with multiple procs.)

Hi Matt,
Ah, I hadn't seen that! This means we should also adapt  the Boozer class to make workers participate in the VMEC evaluation. Apologies for this oversight, I just applied the changes I found worked for SPEC without looking deeper after they seemed to work.
I did find that my changes had an effect: the unittestt that I created would fail for the old code, as would examples/2_intermediate/QH_fixed_resolution_boozer.py (with setting line 19 mpi = MpiPartition(1) and removing line 42 and 44.
Now I understand why I had to remove line 42: a plain evaluation of Quasisymmetry.J() would throw out the workers in line 117, and then let VMEC hang waiting for the workers that never started.
why my change DID work for me is still a bit of a mystery. I think because the first target of the least_squares_problem was vmec.aspect (in example and in test), that would force everyone to participate in the equilibrium calculation each cycle, and then it wouldn't matter that some didn't reach the vmec.run() as it would also be skipped by proc0 who did.
Good catch, and my latest push should fix this.
I intentionally created a new file, as test_integrated_vmec_mpi.py seemed quite specific, and I didn't want to change a name, but I agree that combining the two into test_integrated_mpi.py would be a neater solution.
Sorry I have to run home right now, so I cannot add that change, but I hope to get to it tomorrow or if you see the chance, by all means and thank you.

Hi Matt,
Sorry for the delay on this, I just merged the two tests into one file, which tests parallel simsopt-vmec integration and parallel evaluation of residues and quasisymmetry.
This was the major point needed to run SPEC optimizations massively parallel, so it would be a nice addition to master when all tests pass sucessfully.


Broken link on "getting started on MDN" page

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details

The link "Update API page layout (30 minutes)" on the page Getting started on MDN (under Option 3) refers to a page that doesn't exist.

I've fixed the broken links on this page. Thanks for reporting!


TILA-1609 | Add order by to Applications

Order by filter to applications GraphQL query
Tests for the different ordering types

Refs TILA-1609

Question @nicobav , As you can see in the filtering, I annotated the "applicant" field for filtering with a 3 part database Case clause. My question is, the current "applicant_" fields uses the "user" field but this ordering is more elaborate. Should the "applicant_" field be updated to be as elaborate as this order by?
What I mean is to select the applicant_* values with the same logic.

Question @nicobav , As you can see in the filtering, I annotated the "applicant" field for filtering with a 3 part database Case clause. My question is, the current "applicant__" fields uses the "user" field but this ordering is more elaborate. Should the "applicant__" field be updated to be as elaborate as this order by?
What I mean is to select the applicant_* values with the same logic.

This looks awesome :+1:
I would say that we leave the output data as is.


Font-sizes not working

The font sizes don't seem to work in the editor, they're not working in the demo on your site either:
https://killercodemonkey.github.io/ng-quill/demo.html#Custom font sizes
How would I go about fixing this?
Thanks for the package, really makes working with quill a dream in Angular :+1:

hey i think they are working. this is only a "hacky" result of using multiple editors on one page.
And the custom font-sizes are colliding with the default once i think.
Removed the custom font sizes example and now the font size selection is working again.


Class properties are not deep cloned on extension

It is sometimes annoying that array and object class properties are not deep cloned upon extending the class.  This leads to issues where modifying that property on the subclass also modifies it on the super class.  For example:
class Test
    @test = []

class Foo extends Test

Foo.test is Test.test #=> true

Foo.test.push 'hello world'
Test.test[0] is 'hello world' #=> true

I think that either CoffeeScript should deep clone those objects (probably not), or that this is a case for the extended hook to return.  Here is what I do currently: since I have a class method that uses my class level property, I do something like this:
class Test
    @test = []

    @doSomething: ->
        if @test is @__super__.constructor.test
            @test = @test.slice() # clone it

        # do something...

A dirty hack indeed, and I have to do that on each method that uses @test.  The extended hook would allow me to do that cloning process when the class is extended and avoid this.  For example:
class Test
    @test = []

    @extended: ->
        @test = @test.slice() # clone

    @doSomething: ->
        # do something...

Much cleaner.  Thoughts?

Sorry, Devon; hearing no additional support for this issue, I'm closing it as a duplicate of #841.
In your example, you could just as easily write
class Foo extends Test
  @test = Test.test.slice()

If Test has a lot of children where you want the same behavior, you could put all the code into a single method on Test (say, Test.addChild) and write
class Foo extends Test
  Test.addChild this

Wow... I realize I can do that, but that kind of thing is exactly what I'm trying to avoid.  I'm writing a framework to be used by people other than me and I don't want them to have to call my initialization method every time they extend my class, which will be frequently since it is the base model object in an ORM.  Is there a case against adding the extended hook back?  It seems like a simple change that would make many things easier.

@devongovett: I think the extended hook is on its way back in, I just can't tell you when. There's been almost entirely favourable reactions to it over the past few months.

A very common pattern is "options" found in MooTools Class, jQuery's widget factory, and others.  I'd prefer class did a deep copy of properties as well.
class A

  options:
    foo: 1
    bar: 2

  constructor: (options) ->
    @options = _.extend {}, @options, options

class B extends A

  options:
    foo: 2

  constructor: (options) ->
    super options

You lose options.foo.
Given Jeremy's opinion on deep extending (https://github.com/documentcloud/underscore/issues/162) I'm less than hopeful.

Either way, inherited or injected, its still not as nice looking.  Especially since:
# can drop parens here
options = beget A::options,
  foo: 2

#...

class B extends A

  # can't drop parens here :(
  options: beget A::options,
    foo: 2


Authorities - have an API to CRUD project members.

Better APIs for membership management
(allow list of current members, search by attribute, remove members, etc)

The Rust side is merged and released but not deployed. Some changes are required on the Cloud

#6218


Register Command Implementation

What: Implemented a register command which registers an apex class via EPN to a web store
Why: Store admins will be now able to run through the full workflow of registering their Apex class from their command line rather than via workbench or running custom apex code
How: Command triggers query to retrieve custom apex class from store -> Inserts EPN, uniqueName, apexClass fields into sObject type RegisteredExternalService table -> successfully registers store admin’s class
Tests: 85.50% code coverage

Issues with unsigned/unverified commits. Will create a new PR with a new branch (newRegisterCommand_0804)


WIP: add lint workflow

Fixes https://github.com/appwrite/appwrite/issues/4349

I have commited a few tiny changes:

I update lockfile, helps CI/CD to have proper setup
Ran linter to apply linter changes to existing files

THANK YOU! All changes merged 🥳
Please reach out to me on our Discord server if you would like to claim your Appwrite swags! As a way of saying thank you, we would also love to invite you to join the Appwrite organization on GitHub. Please share your GitHub username with us on Discord.
You can accept the invite by visiting https://github.com/orgs/appwrite/invitation. By joining our team, you will officially be an Appwrite maintainer on GitHub.
You can change your membership visibility settings, so your new Appwrite team membership badge will show up on your personal GitHub profile.
Please feel free to look for more PRs you might be interested in helping with on our long list of Hacktoberfest friendly issues and help make Appwrite better :)


Duplicate Flat Line Alerts

Hello! Thanks for making elastalert - it's amazing.
However, I'm trying to use metricbeats + ELK + elastalert.
I made a flatline config file that alerts to slack, but for some reason I"m getting duplicate alerts.
I've looked at past issues, and I did set a realert field. Furthermore, the duplicate alerts I get are back to back.
The output looks something like this:
INFO:elastalert:Ran System Metric Inputs Are Flagging from 2017-03-08 12:10 IST to 2017-03-08 12:10 IST: 1 query hits, 2 matches, 2 alerts sent
INFO:elastalert:Sleeping for 53 seconds
My Conf File:
es_host: localhost
es_port: 9200
name: System Metric Inputs Are Flagging
type: flatline
index: metricbeat-*
threshold: 2000
timeframe:
minutes: 2
doc_type: "metricsets"
realert:
minutes: 1
use_terms_query: true
query_key: beat.hostname
alert:

"slack"

slack_webhook_url:

"https://hooks.slack.com/services/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"

My back to back slack messages:
12:11]
System Metric Inputs Are Flagging
System Metric Inputs Are Flagging
An abnormally low number of events occurred around 2017-03-08 12:10 IST.
Between 2017-03-08 12:08 IST and 2017-03-08 12:10 IST, there were less than 2000 events.
@timestamp: 2017-03-08T10:10:59.227564Z
beat.hostname: nir-ThinkPad-T430s
count: 27
key: nir-ThinkPad-T430s
num_hits: 1
num_matches: 2
[12:11]
System Metric Inputs Are Flagging
System Metric Inputs Are Flagging
An abnormally low number of events occurred around 2017-03-08 12:10 IST.
Between 2017-03-08 12:08 IST and 2017-03-08 12:10 IST, there were less than 2000 events.
@timestamp: 2017-03-08T10:10:59.227564Z
count: 27
key: nir-ThinkPad-T430s
num_hits: 1
num_matches: 2
I'm having trouble identifying the problem. It has '2 matches', the time frame is 2 minutes, and elastalert is querying elasticsearch every minute - is this the issue? Is that why there are 2 matches and 2 alerts?
Thanks again for your time, and for elastalert :)

You could just set realert to a higher value, like 5 minutes, and that would prevent this from happening.
As for the underlying issue, I'm not completely sure yet. I will try to use these settings and recreate this.


Enforce JSHint in npm test

npm test currently runs JSCS in addition to running the Vows test suite. We should run JSHint too, with something like
jshint --exclude 'public/javascript/libs' bin lib public/javascript routes test

The only reason this can't go in immediately is because the above command currently fails.

We can at least partially enforce this. Here's what needs to be done:

[x] bin/
[ ] lib/
[ ] public/javascript/
[ ] routes/
[ ] test/


Add Reef support

... and default to Quincy. I haven't figured out antsibull-changelog yet.
Closes: #126

Some lint issues


should not load rake when not needed

https://github.com/rollbar/rollbar-gem/blob/v2.12.0/lib/rollbar/plugins/rake.rb#L2
always loads rake, which adds another .1s to our apps load time ... can we make this only patch when rake was already defined ? (for example user is actually running rake)

I think this was added via https://github.com/rollbar/rollbar-gem/pull/485 ... @jondeandres

@grosser thanks for reaching us and open this issue. I'm not sure what to do here. Few customers had problems cause they didn't have Rake loaded, so we decided to try to load it by ourself. I understand your point and we are thinking on this.
We were thinking on something like config.disabled_plugins = ['rake'], but you'd need maybe to detect what to add to that array and it could be complex for the users. On the other side, loading the plugin only if Rake is defined, will break the customer's reports on next gem releases.

I think the correct solution is to silently not enable the plugin ... I don't think many users load rollbar and then load a rake task and then execute it ... it looks like the issue was opened because rollbar was spamming stderr when it should have kept silent ?

@grosser well, we have several customers having long or short rake processes where they use Rollbar. And they want to identify when a rake task failed.
We are thinking on what to do here, thank you very much for your opinion.

... alternatively a disabled_plugins flag would also work ... just sad to see another one of these 'require everything' points ... just noticed because we have some load tracking in our app to notice them ...

It broke the logic of our application: !defined?(::Rake) now always false :(

@KonstantinReido sorry, I didn't get that. Which problem you have or you had?

@KonstantinReido I still don't understand how we broke your application. requiring rake broke your app?

We have code in initializers that shouldn't run during migration so we added next condition if Rails.env.production? && !defined?(::Rake) but after update of rollbar gem now defined?(::Rake) always true.

I'd guess the app checks if defined?(Rake) to see if it is running via
rake or not
On Mon, Sep 19, 2016 at 7:46 AM, Jon<EMAIL_ADDRESS>wrote:

@KonstantinReido https://github.com/KonstantinReido I still don't
understand how we broke your application. requiring rake broke your app?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rollbar/rollbar-gem/issues/524#issuecomment-248014459,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAsZ6tP6z3x839GHqrFKAe8uzJ7-8oEks5qrqBYgaJpZM4J9cSv
.

Hi there, I’m closing out all issues opened before 2018 that haven’t had any activity on them since the start of this year. If this is still an issue for you, please comment here and we can reopen this. Thanks!


Add Auto Comment Feature to Improve Collaboration

Issue Description:
As an active contributor to your open-source project, I believe that implementing an auto-comment feature would greatly enhance collaboration and communication within the project. This feature would automatically generate comments in response to specific events, such as when an issue is opened, a pull request is created, an issue is assigned, or an issue is unassigned.
Feature Details:

When an issue is opened, the auto-comment should greet the author and provide a brief acknowledgement and request for additional context.
When a pull request is opened, the auto-comment should greet the author, express gratitude, and remind them to follow the project's contributing guidelines.
When an issue is closed, the auto-comment should thank the author for their contribution and encourage further engagement.
When an issue is assigned to someone, the auto-comment should notify the assignee and encourage them to start working on it.
When an issue is unassigned from someone, the auto-comment should notify the assignee about the change and suggest reassignment if they are offline.

Benefits:

Improved communication and engagement with contributors.
Provides clear instructions and acknowledgements for various events.
Enhances collaboration by setting expectations and providing reminders.
Reduces manual effort by automating comment generation.

Acceptance Criteria:

The auto-comment feature should be implemented using the "wow-actions/auto-comment" GitHub Action.
Comments should be appropriately customized for each event, mentioning relevant parties and providing the necessary information.
The auto-comment workflow should trigger on the following events: issues opened, pull requests opened, issues closed, issues assigned, and issues unassigned.
The feature should be added to the project's existing GitHub Actions workflow file.

Additional Context:
Feel free to ask any questions or seek clarification regarding the auto-comment feature. I'm excited about contributing to your project and believe this enhancement will greatly benefit its community.

Hey @Nishitbaria , we do not need that much.
Though if possible, can we build a flow to write a message on every PR merge?
If yes, go ahead. Else we do not need that much automation for now.

Yes it's possible

Any update @Nishitbaria?

@nikhil25803
Yes my Work  is almost done I will create PR Checkout

@nikhil25803 I Created My PR Please Review It And Merge it: https://github.com/Clueless-Community/scrape-up/pull/346


use || to silence exit 1 properly and quote fasd init

This PR fixes two mistakes I made in #1602:

when checking for a command || should be use to silence the non-0 exits
fasds --init output needs to be wrapped in quotes, as is documented in their readme :man_facepalming:

My bad y'all!
Fixes #1605

Thanks!


Keyboard shortcut causes Ctrl key to act stuck

Microsoft PowerToys version
0.56.2
Running as admin

[ ] Yes

Area(s) with issue?
Keyboard Manager
Steps to reproduce
Create a keyboard manager shortcut "Ctrl + any key" (specifically "Ctrl", not "Ctrl (Right)" or "Ctrl (Left)"), then trigger this shortcut using right ctrl.
✔ Expected Behavior
The trigger to work with either left ctrl or right ctrl
❌ Actual Behavior
The shortcut worked as expected, but afterward the computer behaved as if the ctrl key was still being held. This behavior continued until left ctrl was pressed and released. The issue only occurred when the shortcut was triggered using right ctrl.
I was able to reproduce this on another computer, the only difference being that right ctrl had to be pressed to correct the issue instead of left ctrl.
Other Software
No response

Hi, We'd like to try to replicate this.
Does it happen everytime?
What specific mapping is being used to cause this?
/needinfo

Yes, it happens every time. I first noticed the issue using “Ctrl+PgUp” and “Ctrl+PgDn” as shortcuts to adjust the volume. After I would trigger the shortcut with right ctrl, links would open in new tabs and keys like S and P would activate the save and print shortcuts, as if ctrl was still being held down. I tested it with Ctrl + various other keys, and it seems to happen no matter what the shortcut is and what action it triggers, even if I set it to “Undefined”.
Out of curiosity, I just tried it with shift and had the same problem. After creating a “Shift+some key” shortcut and triggering it using right shift, everything was typing in all caps as if shift was still being held, until I pressed left shift to “fix” it. So it appears the issue isn’t specifically related to the control key. Once again, it didn’t happen when the shortcut was defined with “Shift (Right)” or “Shift (Left)”, only with “Shift”. In the meantime, to get around the bug, I have defined redundant “Ctrl (Left)+key” and “Ctrl (Right)+key” shortcuts in place of "Ctrl+key" and have had no problems.
Lastly, if it helps, I have been able to reproduce this on both Windows 10 and 11.

That's very helpful!
I'm able to replicate this.
So this happens when you use Ctrl+another key, in the mapping, instead of Ctrl(Left) or Ctrl(Right).
Then if you use Ctrl(Right) as the actual shortcut, it won't register you've actually released the key.
This is likely what's happening.
Thanks a lot for the workaround as well.

Also may be related to:

#16333

Wow, until reading this issue I changed 5 of my keyboards and literally migrated to different OSs to avoid this issue. I started to notice that the issue isn't happening after re-installing Windows for the 3rd time (in 1 a year) and accidentally forgetting to install PowerToys.
I love PowerToys and I hate to see this horrible issue happening and there's very less info available about it. This issue shouldn't be kept in the backlog for over a year and must be looked into.

Came here to report the same, but I'm glad there are already a few issues raised.
I've experienced this in the past weeks, on both Windows 10 and Windows 11 with PowerToys installed, so I assumed this wouldn't be due to Windows updates, but instead due to PowerToys, since there are some functions there that handle keyboard shortcuts.
This seems to be happening with both Shift and Ctrl, sometimes getting stuck, so I have to press them again to "resolve" this issue when that happens.

Wow, until reading this issue I changed 5 of my keyboards and literally migrated to different OSs to avoid this issue. I started to notice that the issue isn't happening after re-installing Windows for the 3rd time (in 1 a year) and accidentally forgetting to install PowerToys.
I love PowerToys and I hate to see this horrible issue happening and there's very less info available about it. This issue shouldn't be kept in the backlog for over a year and must be looked into.

I feel your pain, i have done the same with other un related issues that turn out to be simple fixes. we are martyrs to our cause

still happening September 2024. my ctrl sticks, but at least i now know why. thanks for power toys. they really improve Window quality of life

I've also got this, just rebooted my machine and it's happening instantly.
Interestingly, tapping CTRL shows the "here's where the cursor is" circle animation, but a keyboard shortcut using CTRL doesn't work, and acts as though the CTRL key isn't held at all.

i just downgraded the power toys version to V0.82.0. that fixed it


Add Mobile E2E documentation

Summary

Add mobile E2E documentation

Ticket Link
NA

@saturninoabril Added intro and guide

Thanks for this, @josephbaylon! It's really helpful. I've added quite a few suggestions, but they're mainly around formatting and some rewording. Let me know when it's ready for a second pass; I want to make sure I didn't miss anything. :)

@lindalumitchell @justinegeffen Thanks very much for the meticulous review! I tried to apply all the changes but there were some overlapping so I tried as best as I can to consolidate. Please have another pass at the documentation. Thanks again!

Thanks, @josephbaylon - totally understand about any overlap, I'll sort that out. :)

Oh and I meant to say: In the case of overlap b/w me and Justine, I definitely defer to Justine as our steadfast keeper of the style. 🙇♀

Awesome @justinegeffen and @lindalumitchell !! Attaching the screenshots of each page in case you're interested to see.
@justinegeffen I might request your review again later if @enahum has more change requests. Thanks!

Thanks, @josephbaylon. I meant to mention that I wanted to simplify some changes I made, so apologies for adding additional work in terms of the suggestions. Happy to do another review if needed but it looks good to me. :)

@saturninoabril You mentioned you encountered the deploy failures before. Not sure who to include in this PR to take a look at it. Please take a look and add the necessary personnel. And please merge since I'll be out next week. Thanks!

@enahum Thanks for the reviews. Please take a look again.


[stable/minio] Cannot upgrade deployed chart due to "make-bucket-job" already exists

Is this a request for help?: no (solved by deleting ressources directly in kubernetes)

Is this a BUG REPORT or FEATURE REQUEST? (choose one): bug
Version of Helm and Kubernetes:
Client: &version.Version{SemVer:"v2.10.0", GitCommit:"9ad53aac42165a5fadc6c87be0dea6b115f93090", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.10.0", GitCommit:"9ad53aac42165a5fadc6c87be0dea6b115f93090", GitTreeState:"clean"}

Which chart:
stable/minio
What happened:
Upgrading minio delpoyment fails:
❯ helm upgrade -f minio-values.yml minio-cache stable/minio
Error: UPGRADE FAILED: jobs.batch "minio-cache-make-bucket-job" already exists

What you expected to happen:
Upgrade executed
How to reproduce it (as minimally and precisely as possible):
Deploy stable/minio, then upgrade using a new minio-values.yml:
defaultBucket:
  enabled: true
  ## If enabled, must be a string with length > 0
  name: gitlab-cache
  ## Can be one of none|download|upload|public
  policy: none
  ## Purge if bucket exists already
  purge: false

Anything else we need to know:
Initial deployment was not using minio-values.yml.

Update: it seems that even delete doesn't get rid of the job:
~/htdocs/gitlab-runner
❯ helm delete --purge minio-cache
release "minio-cache" deleted

~/htdocs/gitlab-runner
❯ helm ls --all minio-cache

~/htdocs/gitlab-runner
❯ helm install -f minio-values.yml --name minio-cache stable/minio

Error: jobs.batch "minio-cache-make-bucket-job" already exists

More weirdness. Deployment is happy in the kubeui but fails inconsistently on the command line:
❯ helm install -f minio-values.yml --name minio-cache stable/minio
Error: a release named minio-cache already exists.

❯ helm ls --all minio-cache
NAME            REVISION        UPDATED                         STATUS  CHART           APP VERSION                     NAMESPACE
minio-cache     1               Thu Aug 23 15:44:40 2018        FAILED  minio-1.6.2     RELEASE.2018-07-10T01-42-11Z    default

❯ helm upgrade -f minio-values.yml minio-cache stable/minio
Error: UPGRADE FAILED: "minio-cache" has no deployed releases

Please feel free to close if this is off-topic or more a kubernetes than help problem.

The delete-job pod spits out this error during deletion: Error from server (Forbidden): jobs.batch is forbidden: User "system:serviceaccount:default:default" cannot list jobs.batch in the namespace "default"


[Android] make_apk: Make the code handling icons easier to read.

customize.py is horrible to read, and CustomizeIconByDict() is one
of the biggest offenders.
The first two commits does some clean-up that allows us to get rid of
some checks or moves them to an earlier stage in make_apk.
The last commit rewrites CustomizeIconByDict() to make it easier to
follow, mostly by getting rid of drawable_dict and all the attempts to
be smart by over-manipulating it.

@rakuco Can we close this issue?

Absolutely. Rest in peace, make_apk :-)


Load image from server disk

Hi !
How could we load a default sprite, as it is the case with the megaman sprite on http://www.piskelapp.com/ , which would be stored on the server side ?

Hi. Not sure I understand the request here. If you have an account on piskelapp.com, you can save sprites to your gallery and they are saved on the server.

I would like to use the code on a personal project where I already have sprites on a server. I would like to edit and create new sprite using piskel API.

On the right side of the screen you can click on the Import button (symbol is a folder).

Then you just import the files from your computer.
You might have to move the files from your server to the computer, and they have to be in the format PNG, JPG, BMP or GIF!

Thanks,
I see how I can open files from my computer. My question was more about the API.
For example, in app.js, we have:
layer.addFrame(frame);
I was wondering if it's possible to do something like this:
layer.OpenFile(FilePathOnServer);

As I said earlier, the goal is to be able to do the same thing than the website  http://www.piskelapp.com/ where it opens a megaman sprite on the load up.

So the way it works on piskelapp is that the page contains a serialized sprite and when app.js boots up it deserializes the sprite and loads it in the application, which is probably not what you need here.
Without having more information about the way you embed/use piskel and the format in which your sprites are saved, it's hard to know what would be useful to you here exactly.
The following code snippet is a good example of what needs to be done to load a sprite if it was previously saved/serialized by Piskel.
    // getSerializedSpriteFromServer is for you to figure out
    var serializedPiskel = getSerializedSpriteFromServer(path);

    pskl.utils.serialization.Deserializer.deserialize(serializedPiskel, function (piskel, extra) {
      pskl.app.piskelController.setPiskel(piskel);
      pskl.app.previewController.setFPS(extra.fps);
    });

Now I don't know in which format the sprites are stored on your server. You might have to convert them to something Piskel can process. If there are image spritesheets, have a look at the ImportService, it might help.

Hi !
Thanks for your awnser.
I was thinking to use the piskel format, as I want to eventually save modifications.
I'll take a look at this ! Thanks !

No problem, glad to help!
One last comment, the code snippet I pasted is based on the last version of Piskel. If your fork is a bit old you might have to re-synchronize (we used to have to set the descriptor, but it's now handled by Deserializer.deserialize

Closing for now.


fix for bug #454 on master branch, adds _isHTML condition at HTMLServer.js:151 #567

adds _isHTML condition at line 151 before loading file

sorry for the delay @NYCJacob! This looks right, I'll merge it in, thanks!

@gideonthomas excellent!


Catacombs causing crash

BvB has run across a possible issue with the Gravestone catacombs crashing upon generating in the world.
http://pastebin.com/D2V7v2M2
Can anyone else replicate this.  Similar issue to what happened in TC and the reason they were disabled in that pack.

I have not seen this yet Go,  I have explored 3 different .5.4 worlds now.

Since we have not had any more reports of this I will close this for now.  It may be a non-issue going forward depending on the changes in the pack that are planned


Running ngen on CONUS with Routing: Build, Realization Configs, Routing Config, and Documentation

This PR adds several realization config files and the routing config file for running ngen with routing on CONUS. It also provides specific documentation on how to build and run MPI jobs with routing in ngen framework,
Additions
Realization config files for runing routing after noah-owp-modular, running routing after noah+pet+cfe, running routing
after noah+pet+smp+sft+cfe, all on CONUS.
Also the configuration file for running routing on CONUS: routing_config_CONUS.yaml.
Updated documentation.
Removals

Changes

Testing
Running ngen tests with all 3 realization configs.
Screenshots
Notes
This PR should be merged after PR#775 and PR#794. Although no real conflict, it is better for understanding the construction of the realization configs and the documentation.
Todos

Checklist

[x ] PR has an informative and human-readable title
[x ] Changes are limited to a single goal (no scope creep)
[x ] Code can be automatically merged (no conflicts)
[x ] Code follows project standards (link if applicable)
[x ] Passes all existing automated tests
[x ] Any change in functionality is tested
[x ] New functions are documented (with a description, list of inputs, and expected output)
[x ] Placeholder code is flagged / future todos are captured in comments
[x ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[x ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist (automated report can be put here)

Target Environment support

[x ] Linux

Thanks for the review. Will implement the suggested changes.
On Fri, May 3, 2024 at 10:33 AM Justin Singh-M. - NOAA <
@.***> wrote:

@.**** requested changes on this pull request.
In doc/NextGen_ON_CONUS.md
https://github.com/NOAA-OWP/ngen/pull/805#discussion_r1589357236:

Run Computation with Topmodel

To be added
Run Computation with Routing
-To be added
+To run computation on CONUS with routing, we need to build the executable with routing option turned on. This can be done using the build script displayed in the Build the Executable and set -DNGEN_WITH_ROUTING:BOOL=ON. Make sure that Python is also on, and you can run the script to build the ngen executable.
⬇ Suggested change
-To run computation on CONUS with routing, we need to build the executable with routing option turned on. This can be done using the build script displayed in the Build the Executable and set -DNGEN_WITH_ROUTING:BOOL=ON. Make sure that Python is also on, and you can run the script to build the ngen executable.
+To run computation on CONUS with routing, we need to build the executable with the routing option turned on. This can be done using the build script displayed in the Build the Executable section, and ensure both -DNGEN_WITH_PYTHON:BOOL=ON and -DNGEN_WITH_ROUTING:BOOL=ON are enabled. Then, you can run the script to build the ngen executable.

In doc/NextGen_ON_CONUS.md
https://github.com/NOAA-OWP/ngen/pull/805#discussion_r1589359789:

Run Computation with Topmodel

To be added
Run Computation with Routing
-To be added
+To run computation on CONUS with routing, we need to build the executable with routing option turned on. This can be done using the build script displayed in the Build the Executable and set -DNGEN_WITH_ROUTING:BOOL=ON. Make sure that Python is also on, and you can run the script to build the ngen executable.
+
+You also need to build the t-route submodule. First, as the automatically downloaded t-route is out of date, to get the latest version, you need to remove the old t-route and run git clone https://github.com/NOAA-OWP/t-route https://github.com/NOAA-OWP/t-route in the extern directory. For building t-route in general, we refer user to the documentation PYTHON_ROUTING.md for essential details. We just want to add some additional discussion here to make the process easier. Note that there are more than one ways to build the t-route. Do cd t-route, then run:
⬇ Suggested change
-You also need to build the t-route submodule. First, as the automatically downloaded t-route is out of date, to get the latest version, you need to remove the old t-route and run git clone https://github.com/NOAA-OWP/t-route https://github.com/NOAA-OWP/t-route in the extern directory. For building t-route in general, we refer user to the documentation PYTHON_ROUTING.md for essential details. We just want to add some additional discussion here to make the process easier. Note that there are more than one ways to build the t-route. Do cd t-route, then run:
+You also need to build the t-route submodule. First, as the vendored t-route submodule is out of date, to get the latest version, you need to remove the old t-route and run git clone https://github.com/NOAA-OWP/t-route https://github.com/NOAA-OWP/t-route in the extern directory. For building t-route in general, we refer to the documentation PYTHON_ROUTING.md for essential details. We just want to add some additional discussion here to make the process easier. Note that there is more than one way to build t-route. Do cd t-route, then run:

In doc/NextGen_ON_CONUS.md
https://github.com/NOAA-OWP/ngen/pull/805#discussion_r1589362110:

Run Computation with Topmodel

To be added
Run Computation with Routing
-To be added
+To run computation on CONUS with routing, we need to build the executable with routing option turned on. This can be done using the build script displayed in the Build the Executable and set -DNGEN_WITH_ROUTING:BOOL=ON. Make sure that Python is also on, and you can run the script to build the ngen executable.
+
+You also need to build the t-route submodule. First, as the automatically downloaded t-route is out of date, to get the latest version, you need to remove the old t-route and run git clone https://github.com/NOAA-OWP/t-route https://github.com/NOAA-OWP/t-route in the extern directory. For building t-route in general, we refer user to the documentation PYTHON_ROUTING.md for essential details. We just want to add some additional discussion here to make the process easier. Note that there are more than one ways to build the t-route. Do cd t-route, then run:
+
+ +FC=mpif90 NETCDFINC=<path-to-netcdf-include-directory> ./compiler.sh +
+
+This will let you to use the NetCDF library that you want to build t-route. Alternatively, you can edit the compiler.sh file, note these comment lines in the script:
⬇ Suggested change
-This will let you to use the NetCDF library that you want to build t-route. Alternatively, you can edit the compiler.sh file, note these comment lines in the script:
+This will let you use the NetCDF library that you want to build t-route. Alternatively, you can edit the compiler.sh file, note these comment lines in the script:

In doc/NextGen_ON_CONUS.md
https://github.com/NOAA-OWP/ngen/pull/805#discussion_r1589363394:

+ +FC=mpif90 NETCDFINC=<path-to-netcdf-include-directory> ./compiler.sh +
+
+This will let you to use the NetCDF library that you want to build t-route. Alternatively, you can edit the compiler.sh file, note these comment lines in the script:
+
+ +#if you have custom dynamic library paths, uncomment below and export them +#export LD_LIBRARY_PATHS=<paths>:$LD_LIBRARY_PATHS +
+
+You can uncomment the second line above and that put your NetCDF library in the path. In addition, change this line:
+
+```

export NETCDFINC=/usr/include/openmpi-x86_64/

⬇ Suggested change

export NETCDFINC=/usr/include/openmpi-x86_64/
+export NETCDFINC=/usr/include/openmpi-x86_64/

In doc/NextGen_ON_CONUS.md
https://github.com/NOAA-OWP/ngen/pull/805#discussion_r1589363540:

+ +#if you have custom dynamic library paths, uncomment below and export them +#export LD_LIBRARY_PATHS=<paths>:$LD_LIBRARY_PATHS +
+
+You can uncomment the second line above and that put your NetCDF library in the path. In addition, change this line:
+
+```

export NETCDFINC=/usr/include/openmpi-x86_64/
+```

+to the path to your NetCDF include directory, i.e.
+
+```

export NETCDFINC=

⬇ Suggested change

export NETCDFINC=
+export NETCDFINC=

In doc/NextGen_ON_CONUS.md
https://github.com/NOAA-OWP/ngen/pull/805#discussion_r1589365333:

export NETCDFINC=/usr/include/openmpi-x86_64/
+```

+to the path to your NetCDF include directory, i.e.
+
+```

export NETCDFINC=
+```

+Then, run the command:
+
+ +FC=mpif90 ./compiler.sh +
+
+After successfully building the t-route, you can run ngen with routing. Note that we have several realization configuration files and the routing_config_CONUS.yaml file for running ngen with routing. The realization configuration file and routing_config_CONUS.yaml specify where the input and output files are. For routing, we assume the existence of a stream_output_dir directory for writing output files. You need to do mkdir stream_output_dir before running ngen. With that, we can run an example with the command:
⬇ Suggested change
-After successfully building the t-route, you can run ngen with routing. Note that we have several realization configuration files and the routing_config_CONUS.yaml file for running ngen with routing. The realization configuration file and routing_config_CONUS.yaml specify where the input and output files are. For routing, we assume the existence of a stream_output_dir directory for writing output files. You need to do mkdir stream_output_dir before running ngen. With that, we can run an example with the command:
+After successfully building t-route, you can run ngen with routing. Note that we have several realization configuration files and the routing_config_CONUS.yaml file for running ngen with routing. The realization configuration file and routing_config_CONUS.yaml specify where the input and output files are. For routing, we assume the existence of a stream_output_dir directory for writing output files. You need to do mkdir stream_output_dir before running ngen. With that, we can run an example with the command:

In doc/NextGen_ON_CONUS.md
https://github.com/NOAA-OWP/ngen/pull/805#discussion_r1589366190:

+* trt - abrevitation for t-route
+* For all other abreviations, see Resource Usage.

⬇ Suggested change
-* trt - abrevitation for t-route
-* For all other abreviations, see Resource Usage.
+* trt - abbreviation for t-route
+* For all other abbreviations, see Resource Usage.
—
Reply to this email directly, view it on GitHub
https://github.com/NOAA-OWP/ngen/pull/805#pullrequestreview-2038405000,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACA4SRLNVWBTLEXCMRBC2PDZAOU5PAVCNFSM6AAAAABG3DTVQ2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMZYGQYDKMBQGA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Console on disable during server shutdown

Describe the Bug
Console error on plugin disable during server shutdown.
https://paste.gg/p/Momshroom/07f69e2e6ad64e459e1ebc4ac609eeab
Expected Behavior

No errors :3
Plugin Version

2.1.1
Towny Version
<IP_ADDRESS>
Pl3xMap Version
1.20-441
Server Software Version

Paper 1.20.1 build 43
Additional Comments
Thanks for the cool plugin!

Hi,
Thanks for using the plugin! Just to make sure, are you using MapTowny 2.2.0? Only MapTowny v2.2.0 supports Pl3xMap v3.

On Fri, Jun 23, 2023 at 11:37 PM silverwolfg11 @.***>
wrote:

Hi,
Thanks for using the plugin! Just to make sure, are you using MapTowny
2.2.0? Only MapTowny v2.2.0 supports Pl3xMap v3.
—
Reply to this email directly, view it on GitHub
https://github.com/TownyAdvanced/MapTowny/issues/46#issuecomment-1605263166,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEATM2D4UBSDBRVFH2EXMILXMZVKJANCNFSM6AAAAAAZSH6BOA
.
You are receiving this because you authored the thread.Message ID:
@.***>

I’m using 2.2.1.  Llmdl compiled it for smallsansserif on the towny discord
and I also downloaded it.

Clarifying that I'm using it compiled from the latest commits.  There wasn't a released binary with the Pl3xMap 3 support

Hi,
I looked into the issue, but unfortunately I'm not able to reproduce it. The code-related portion of the error also looks correct, so I'm not even sure how the error could occur in the first place.
I think the next steps would be trying to find some sort of pattern that would lead to reproducing the issue. Does the issue occur on every shutdown so far?
I also noticed that there are new Pl3xMap updates available, so maybe that could potentially fix the issue? I'm not sure if you want to give that a try.
For context, I'm running / testing on the environment:

Paper-538 (1.19.4)
Pl3xMap-1.19.4-435.jar
MapTowny 2.2.0 release

I don't think the differences in environments would cause the issue since MapTowny is compiled against Pl3xMap-1.20.1-446.

This seems to be fixed with the release map towny 2.2.0 and pl3xMap updates.  I'm not sure which fixed it, but thank you for looking into it.


Update calling-plans-for-office-365.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/8331
Removed screenshot as does not reflect correctly in licensing.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

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


DiagonalOperator for dot operators?

A .* array operation can currently be implemented by reshaping to a column and then applying a diagonal matrix cast as DiagonalOperator.
Reshaping uses memory for large arrays, so I wonder if there is scope to handle dot operations in this API since they are equivalent to DiagonalOperator?

DiagonalOperator uses vec which is in place, so there shouldn't be any memory allocation, besides 64 bytes for the integer tuple that stores size -- negligible for most applications.
if you see a substantial memory footprint with DiagonalOperator and mul! or ldiv! let me know and we'll sort this out. Alternatively if you want to eliminate the size allocation, let me know. there are some view tricks that we can play

as a rule, no SciMLOperator should allocate when used as in

cache_operator(L, u) # allocate cache

# allocation free
mul!(v, L, u) # apply
ldiv!(v, L, u) # invert
L(v, u, p, t) # apply

@AshtonSBradley  @ChrisRackauckas close issue?


Add PhoneType in mme-exporter

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?


bootique-jooq-codegen

Module witch generates JOOQ classes from provided DataSources.

So idea was to create module, which can be used as cli tool for code-generation. Like in this example, but in Bootique way.


CTCP-617 Adding auditcontroller, routing

I realise there's going to be a deep irony in what I'm saying here, but can you flip auditing -> enabled to false here? We'll only turn it on in the appropriate environments (https://github.com/hmrc/app-config-base/blob/57df7d3e714f2d4cd65f8de4efcc843e7c52255e/transit-movements-auditing.conf#L9), else we're going to have loads of errors when running this.
Otherwise I don't see anything else of concern, unless we want to do that integration test for binding (though there is a question there as to whether that's just testing the framwork at that point)

Ok, will leave the integration test out for now.


Error: "More than one git repo URL template matched URL"

We recently tried to upgrade from Edgware.SR5 to Greenwich SR3. ConfigServer reports the following error for certain requests:

More than one git repo URL template matched URL

The error is coming from the HttpClientConfigurableHttpConnectionFactory.lookupHttpClientBuilder method. With Greenwich SR3, a bugfix was implemented in that class based on a support ticket my team opened. However, it appears it was only a partial fix.
We have a bootstrap similar to this:
spring:
  cloud:
    config:
      server:
        git:
          uri: https://blahblah/v1/repos/vvv_configs-{application}
          proxy:
            https:
              host: ${proxyuri}
              port: '8080'

          repos:

            WebApp:
              pattern: webapp
              searchPaths: SharedDomains/*, {application}
              basedir: ${basedirParent}\\baseWebApp
              uri: https://blahblah/v1/repos/vvv_configs-webapp
              proxy:
                https:
                  host: ${proxyuri}
                  port: '8080'

The problem is that if you call configserver for application "webapp", the lookupHttpClientBuilder method finds both uris shown in the sample bootstrap above. The lookupHttpClientBuilder expected to find one and only one. The service will continue on to process the request, but logs error:

More than one git repo URL template matched URL: https://blahblah/v1/repos/vvv_configs-webapp/info/refs?service=git-upload-pack, proxy and skipSslValidation config won't be applied. Matched templates: https://blahblah/v1/repos/vvv_configs-{application}, https://blahblah/v1/repos/vvv_configs-webapp

We need to use the proxy, so this won't work for us.
This did not happen with Finchley. (The HttpClientConfigurableHttpConnectionFactory class did not exist in that release.)

I wanted to first relay the issue and request that it be fixed.
Short-term, I am trying to find a workaround. I am hoping you can help. We have a strong need to upgrade to the latest release very soon.

Workaround:
For a workaround, I tried adding a default repo entry to the end of the bootstrap and removing the default git entry, as shown below. Unsurprisingly, the service won't even start if you don't have spring.cloud.config.server.git.uri defined. So this did not work:
spring:
  cloud:
    config:
      server:
        git:
          repos:
            WebApp:
              pattern: webapp
              searchPaths: SharedDomains/*, {application}
              uri: https://blahblah/v1/repos/vvv_configs-webapp
              proxy:
                https:
                  host: ${proxyuri}
                  port: '8080'

            Default:
              pattern: '*'
              uri: https://blahblah/v1/repos/vvv_configs-{application}
              proxy:
                https:
                  host: ${proxyuri}
                  port: '8080'

We could add the spring.cloud.config.server.git.uri back in to point to just one repo instead of using {application} placeholder and have the Default for everything else. But for that one repo specified on spring.cloud.config.server.git.uri, we'll get the error, so that won't work. We could put a dummy repo in for spring.cloud.config.server.git.uri, but that is too much of a hack for a long-term solution.
Can you think of a way for us to configure repos so that only one repo url is ever found? Is there a way to specify the Default repo as in my example, but in addition to pattern, add an exclusion to exclude the repo present as the default? Something like the example below. In that example, note:

The spring.cloud.config.server.git.uri is set to vvv_configs-authenticationAccounts instead of vvv_configs-{application}

The Default repo has a patternExclude (which isn't a real parameter).

spring:
  cloud:
    config:
      server:
        git:
          uri: https://blahblah/v1/repos/vvv_configs-authenticationAccounts
          proxy:
            https:
              host: ${proxyuri}
              port: '8080'

          repos:

            WebApp:
              pattern: webapp
              searchPaths: SharedDomains/*, {application}
              uri: https://blahblah/v1/repos/vvv_configs-webapp
              proxy:
                https:
                  host: ${proxyuri}
                  port: '8080'

            Default:
              pattern: '*'
              patternExclude: authenticationAccounts
              uri: https://blahblah/v1/repos/vvv_configs-{application}
              proxy:
                https:
                  host: ${proxyuri}
                  port: '8080'

Can pattern be regex, or is asterisk for wildcard the only matching that is supported?
Note that eliminating having a token in the uri (vvv_configs-{application}) and explicitly defining a repo entry for every service and app is not a feasible workaround for us. We have many dozens (100s?) of apps and services. It's not feasible to add a repo entry into the bootstrap for each of them and also have to continue adding for every new service we create.

My half-baked proposed workaround does not appear as if it will work. I can't envision any workaround for this.
The reason is that the lookupHttpClientBuilder method does not track which repo entry in the bootstrap was matched. It just maintains a list of all the repos that were configured and tries to do a string match against all of them. So, no matter where I specify the uri with the {application} placeholder, it seems lookupHttpClientBuilder will always match against that and find multiple URIs for any application that has its own git repo definition.
Shouldn't the lookupHttpClientBuilder method honor the repo configuration in the bootstrap? It seems like a design flaw to me.

A workaround will probably involve creating your own implementation.
Then you need to create a bean that looks like
		@Bean
		@Primary
		public ConfigurableHttpConnectionFactory myHttpClientConnectionFactory() {
			return new MyCustomHttpClientConfigurableHttpConnectionFactory();
		}

Thanks for the quick response! Yes, it's kind of looking like that is the only option.
Is it feasible to override with this custom bean? I don't see how to inject my own bean when EnvironmentRepositoryConfiguration creates MultipleJGitEnvironmentRepositoryFactory.

It's just a bean AFAICT

Got it. Thought there was something special about that one for some reason.

Workaround Info
As suggested, I was able to work around this by injecting my own implementation of the ConfigurableHttpConnectionFactory interface.
For anyone else looking for a workaround, here are some details:

My implementation of the ConfigurableHttpConnectionFactory is a copy of the HttpClientConfigurableHttpConnectionFactory with lookupHttpClientBuilder modified. Instead of trying to match against any URL, including those with placeholders, it first searches for an exact match. Only if no exact match is found does it then consider urls with placeholders.
To inject my implementation, I override the bean in EnvironmentRepositoryConfiguration.JGitHttpClientConfig.
On my class which defines my bean, I have the following. (I only just learned details about spring boot auto-config, so I can't guarantee these are optimal.)

@Primary
@Configuration
@Import({EnvironmentRepositoryConfiguration.class})

To override a bean definition, you may need to add this to your project: spring.main.allow-bean-definition-overriding: true

I don't know if my updated HttpClientConfigurableHttpConnectionFactory could be considered for a permanent fix. It may be that for some teams, finding both an exact match and a placeholder match should actually be treated as an error instead of ignoring the placeholder as my implementation is doing.
Note in case it helps: I found a pull request which describes why this class is treating multiple matches as an error, but it looks like the intention was to catch the case where there were matches against multiple URLs that had placeholders (not one exact match and one placeholder match).
Should I attach my implementation of the connection factory? (It hasn't gone through code review or thorough testing yet, though all my local testing passed.)

I forgot to list this step, which is needed for the Workaround:

Add your class that defines your "overriding" bean to spring.factories file.

I create a sample project demonstrating the issue: https://github.com/marnee01/vts-spring-config-server-1480
Instructions are in the README file at the root of the project (displayed on the main page by default).
Note that the effect of the error is that proxy settings are not being read from the bootstrap file. But I can’t demonstrate that, since putting our proxy settings in that sample app won’t work of course. But this project does replicate the root problem.
(I also have submitted a support ticket and send the link to the project on that ticket.)

@marnee01 since your workaround above fixes the issue would you be able to submit a PR with the workaround so we can incorporate the fix?

The workaround is not a fix, it's just a workaround. It wouldn't fix https://github.com/spring-cloud/spring-cloud-config/issues/1482, which is closely related.

@marnee01
#1482 is a separate issue so lets not combine the two
You stated that

My implementation of the ConfigurableHttpConnectionFactory is a copy of the HttpClientConfigurableHttpConnectionFactory with lookupHttpClientBuilder modified. Instead of trying to match against any URL, including those with placeholders, it first searches for an exact match. Only if no exact match is found does it then consider urls with placeholders.

What I am suggesting is that the PR basically modify HttpClientConfigurableHttpConnectionFactory.lookupHttpClientBuilder with the logic from your workaround.  Would that not fix this issue?

#1482 is a separate issue so lets not combine the two

They have the same underlying cause, though.
It would fix this issue for me. As I said above:

I don't know if my updated HttpClientConfigurableHttpConnectionFactory could be considered for a permanent fix. It may be that for some teams, finding both an exact match and a placeholder match should actually be treated as an error instead of ignoring the placeholder as my implementation is doing.

And I wrote the above before I realized there was a more fundamental issue with the design of the class, which is why this issue is related to the one documented in #1482.
I won't send a pull request, because I'm not going to get set up with the project, figure out maven, figure out how to test the changes that are made directly to the HttpClientConfigurableHttpConnectionFactory class, etc. for something that appears to me to be a band-aid. I don't want to be responsible for breaking someone else's software if they consider this "fix" to instead be a "regression".
I've attached my file. You can do a "diff" against HttpClientConfigurableHttpConnectionFactory to see what I changed.
VTSHttpClientConfigurableHttpConnectionFactory.zip

Closed via  c7ab0f3


iframe p5 sketch loading issue

The p5 sketch in the iframe doesn't load when coming back from a Docisy page.

Fixed in #10
Preview Deployment

@alanvww Thank you! This is fantastic. Can you apply max-width: 800px for the iframe module?

@MOQN I just tried a little more time with my deploy preview and I believe the sketch is loading correctly when the user go back from the docsify page now.(It didn't fail on my device :/) But for the problem in "open in new page", it shows the same error that

DOMException: Failed to execute 'drawImage' on 'CanvasRenderingContext2D': The image argument is a canvas element with a width or height of 0.

This is actually quite interesting and I would like to explore it more besides working on the content. In the meantime, maybe we can try to review the p5 sketch code together?

@alanvww Thank you! Absolutely. Please feel free to look at this code. I got an idea for solving it. Let me explore it and share the details soon.

The hero sketch was newly developed with @XiaozaoWang's hand sketches.


Micropub endpoint: Try to be less clever deriving text and html JF2 values

Previously, this endpoint accounted for the following cases for an incoming JF2 content property:

HTML being provided for content.text
HTML being provided for content

This may be a case of over engineering:

If plaintext contains HTML, this should be considered intentional. Markdown can include inline HTML elements which don’t have equivalents in Markdown (i.e. <cite>).

If plaintext is entirely HTML, that’s an issue with the Micropub client: it should send content using the correct Microformats representation (which is also content.html), and therefore an issue should be filled against that client.

This PR:

removes any attempt to detect HTML in plaintext and convert it to Markdown
adds missing test for deriving content.text from sole content.html value
adds support for keeping <cite>, <ins> and <del> in any HTML converted to text
adds inline <cite>, <ins> and <del> HTML elements to test fixtures

cc @sentience

It does concern me that a field named "plaintext" might contain something other than plain text to be displayed literally by the site. Is sending Markdown in the plaintext field common and expected in Micropub, or is it more correct to send Markdown (as virtually a superset of HTML) in the HTML field?

Looks like this was discussed but with no clear outcome in 2019: https://indieweb.org/Micropub-brainstorming#Additional_data_formats


Unable to install web extension

Does this issue occur when all extensions are disabled?: N/A

VS Code Version:  1.69.2
OS Version: Web
Browser: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Firefox/102.0
(I have also replicated this on iOS and Edge)

Steps to Reproduce:

Open vscode.dev
Search for "Esbonio" in the marketplace
Click "Install"

I (thought I'd) recently added web extension support to my VSCode extension Esbonio.
Before publishing the extension I tested it in a browser by following this guide and everything seemed to work as expected.
The VSCode UI also seems to agree with me that this is a valid web extension as there is no warning icon and an Install button is presented - however clicking install fails with the dialog saying it is not a valid web extension (as shown in the screenshot above).
If I had to guess what was wrong it would be the fact the extension is also an extension pack that depends on an extension that is not a valid web extension. But there is nothing in the UI that tells me that is the case and it seems to contradict the web extension documentation which says the user should still be able to install it anyway.
Any help in diagnosing this would be appreciated :smile:

Note: I've pushed an update removing the extensionPack field from the extension's package.json and it now installs correctly, so this issue does not affect the Esbonio extension anymore.
However, I did some digging and I think I found what was causing the behavior I was seeing above
https://github.com/microsoft/vscode/blob/b3dc3301dd22d9695d5e5bdc69e5e23a6db1be58/src/vs/workbench/services/extensionManagement/common/extensionManagementService.ts#L428-L439
It looks like installation is blocked if every extension declared in an extension pack has no web support - which makes sense if the parent extension doesn't provide any features of its own.
Unfortunately, it also prevents extensions like Esbonio which also provide their own features from being installed as this banner is never shown to the user.

This seems to work now. Hence closing this.


Botframework channel. Change type and rename bot_id

Proposed changes:

Change type and rename bot_id

Status (please check what you already did):

[x] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog

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

Thank you @tupac56 !


Move logic to collect DB snapshots into the migrations module

Wanna reuse this in migrations plugin stuff

Needs squashing


Clicking on question should refresh it

Currently, when viewing all questions, clicking on a question doesn't refresh it. They should be refreshed.

Fixed


Issue 7: Add Support for Processing Multiple Files and Folders

Currently, the tool can process a single file at a time. We are adding support for processing multiple files or directories as input. If a directory is provided, the tool will recursively search through the directory and process all files it finds. The changes required to implement this feature are as follows:
Command-line Argument Parsing:
Update the argument parser to accept multiple file and directory paths.
Distinguish between files and directories by checking each input path.
File Handling:
Add logic to handle directories by recursively finding all files in the directory.
Ensure that each file is processed individually, similar to how single files are currently handled.
Output:
For each file processed, the results should either be printed or written to an output file (if specified).
If multiple files are processed and the user specifies an output file, all results should be appended to that file.
Error Handling:
If a directory or file path is invalid, an error should be returned, and the program should exit with an appropriate error code.
Changes:
main.py: Modify the command-line argument handling, add directory traversal logic, and update file processing.

Issue #7: Support for multiple files and directories has been implemented in commit d3098655f97c4185e4dbdac2fb37b3c99583573a. The tool now processes multiple file paths and directories recursively, allowing users to analyze entire codebases effortlessly.
Closed by commit: d3098655f97c4185e4dbdac2fb37b3c99583573a


Different UserId for ConversationUpdate and Message

The webchat is hosted using standard code to define the user id as below,
var user = {
    id: params["userid"] || "myuser",
    name: params["username"] || "You"
};

When the conversationUpdate event is triggered, the activity.From.Id is something cryptic (like Kq3YtJ7KkL7upmnPpzke) instead of myuser until the user sends a message to the bot.
Once the user sends a message to the bot, activity.From.Id gets assigned to myuser. Is this because ConversationUpdate is a system message. If so, in the emulator, for both system message and normal activities, we get default-user whereas for WebChat, the behavior is different.

Duplicate of https://github.com/Microsoft/BotFramework-WebChat/issues/536 - see there for a workaround using backchannel.
@danmarshall this is probably our most frequently asked question. It would be great to update the docs to talk about how to get around this specific shortcoming of Direct Line. @dandriscoll FYI.


#669  building shake package  panic FIX - working but not understandable solution

This seems to fix the compilation problem (file compiles) - I do not know if it works correctly then. I do not have example.
With this fix all my test projects work and it should work not worse than before fixes than in #603 .
The problem is - this works not as we've agreed. This algorithm ignores last  / non matching cgLocs (as it was done before #603) and removes void args  just from the middle of args - not removing "corresponding" cgLoc.
When I changed it to a proper functionality (my commit https://github.com/jarekratajski/eta/commit/dceadce52fdbee3c959586fb3e7913cf102f51be)
even simple programs started to fail.
I create this PR in order to give "quick fix" - as I  will not have time for eta in next 3 days.

Hi @jarekratajski, thanks for the temporary patch. I will leave this PR open and let you find a better solution to this. My only worry is whether other things will break inadvertently with this change, so let's take this slow.

@jarekratajski Will you have time to implement the changes I suggested in the comments to #669?

Hi!
I will take a look   in next 6-7 hours and give some info.
On Mon, Mar 12, 2018 at 6:02 AM, Rahul Muttineni<EMAIL_ADDRESS>wrote:

@jarekratajski https://github.com/jarekratajski Will you have time to
implement the changes I suggested in the comments to #669
https://github.com/typelead/eta/issues/669?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/typelead/eta/pull/676#issuecomment-372194336, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGBjCH_IUWacK3R1XY05Ttu5BgnMOKBiks5tdgFIgaJpZM4SO39I
.

--
Jaroslaw Ratajski

Ok so I took some time to debug this re-inserting those extra traceCg's you've added. Basically what's happening is cgLocs only consists of non-void locations, while args can consist of void locations. What we should do is filter out the void args and then zip the filtered args with the cgLocs and pass it into dependencies function. I have tested this locally and it works.
Your solution in this pull request will work in 99% of the cases and the reason why it works is that it just so happens the the void arg will be the last element of args, so cgLocs (without the void location) will constrain it to the right size when you do zip.
If you have time to finish up the last bit mentioned above, let me know.

@jarekratajski I'm going ahead with merging this since we have a release coming up.

For reference, these were the minor changes required to make this correct in every case:
https://github.com/typelead/eta/commit/061df4af322df5031bbfffd6b6fd6b6288269c50


consider dropping sugar for (?=\R?\z)

This was brought up briefly during the presentation of this proposal to committee for stage 1.
I'm never going to remember which of \z or \Z is the simple anchor and which has the additional line terminator handling. I will have to look it up every time. If one of them is invalid, it will make it easier for me to find out via editor support, tooling, or even runtime errors. If they're both valid but with slightly different semantics, I probably won't even notice I've chosen the wrong one. I don't think this is worth it for the minor convenience.

I'm worried about being arbitrarily different from other engines as it could be hazardous for developers who work in multiple languages, or are coming to JS from another language. I'd be fine with dropping \Z (the form that allows an optional trailing newline), as long as we are careful not to introduce some other meaning for \Z in the future (so as not to collide with RegExp syntax in other languages).

Stage 2 consensus was to advance \A and \z but only reserve \Z.


feat: update github code links in manual to version branch

Fixes: https://github.com/denoland/deno_website2/issues/1527

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

LGTM, thanks. Now we just have to go through denoland/deno/docs and update the references there.

Here: https://github.com/denoland/deno/pull/7710
One issue with this is that code links will not be browse-sable from Markdown files.

One issue with this is that code links will not be browse-sable from Markdown files.

:man_shrugging: - that is already the case with $STD_VERSION


MCF Browser Switch to Typescript

This PR involves the following changes:

Resolving lint errors
Switching the project over to TypeScript
Fixing the parsing table bug for MCF files

Most of the changes should be formatting changes, type annotations, or type definitions to be used for annotating later.

ptal, I resolved all merge conflicts and removed all instances of console.log.


Discussion: Improve Handling

Type of Report
Feature Request /Discussion
Actual behaviour
Currently a lot of workarounds are needed to get this thing going.
The content is in the root directory of the repo.
Expected behaviour
A rel-ayouting and some minor work can improve the handling:
README.md                  # The description/README.md for this repo
.editorconfig                   # The repo specific .editorconfig
.gitignore                        # The repo specific .gitignore

docs                               # The regular docs will be placed here
tests                               # The tests for this exact repository will be placed here

dist                                 # The skeleton (distributed) will be placed here
   README.md.j2           # Readme.md template for new roles
  .editorconfig                 # The editorconfig for new roles
  .gitignore                      # .gitignore for new repos
  .travis.yml.j2                # Template for new travis files

  docs                             # The docs for new roles will be placed here
  tests                             # Templates for tests
  meta
  tasks

I think you got the idea ^^
What do you think about this? From my perspective it will remove the workarounds, allow tests for the repo itself (including a simulated ansible-galaxy init) and can help to keep it up-to-date. One problem will be there. Updating the repository can cause a lot of copy-pasting, because it can happen that multiple files must be touched.

If the above is ok from your side, I'll start the work and open a PR

Not sure about the details, but I like the idea.
Please proceed :)
Daniel S<EMAIL_ADDRESS>schrieb am Sa., 3. Feb. 2018, 09:56:

If the above is ok from your side, I'll start the work and open a PR
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/while-true-do/ansible-galaxy-skeleton/issues/11#issuecomment-362791721,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIA9lIyLDutZGeBZ05UOCAVos_OT2eBUks5tRB8sgaJpZM4R4GZp
.

starting with this in https://github.com/while-true-do/ansible-galaxy-skeleton/tree/fb-daniel-wtd