Error when loading corpus

Hi,
i am looking forward to use this library. I like the opportunity to plot the Greene, Arun and Brunet-methods.
Today i wanted to test the library (in Python 2.7). I installed via pip.
pip install tom_lib
When i run the example code
corpus = Corpus(source_file_path='input/egc_lemmatized.csv',                 language='french',                 vectorization=vectorization,                 max_relative_frequency=max_tf,                 min_absolute_frequency=min_tf,                 preprocessor=preprocessor)
from topic_model.py
 i get a KeyError.
pandas/index.pyx in pandas.index.IndexEngine.get_loc (pandas/index.c:4154)() pandas/index.pyx in pandas.index.IndexEngine.get_loc (pandas/index.c:4018)() pandas/hashtable.pyx in pandas.hashtable.PyObjectHashTable.get_item (pandas/hashtable.c:12368)() pandas/hashtable.pyx in pandas.hashtable.PyObjectHashTable.get_item (pandas/hashtable.c:12322)() KeyError: 'full_content'
The example csv-file from this repository is in a directory called input.
Am i missing something?
I'm looking forward to hearing from you.
Python Version?
Also i think the source code is not intended by used with Python 3 at the moment? On pypi it says:

TOM (TOpic Modeling) is a Python 2.7 library for topic modeling and browsing.

on GitHub it says

TOM (TOpic Modeling) is a Python 3 library for topic modeling and browsing, licensed under the MIT license.

When i try to run the example scripts in Python 3, it gives a SyntaxError:
  File "...python3.5/site-packages/tom_lib/utils.py", line 13     print matrix[i, :]

Hi,
Thank you for this feedback!
Key error
The latest versions of TOM (0.2.x) assume a different structure for input CSV files, in particular the 'full_content' column has been renamed to 'text' while 'short_content' has been renamed to 'title'.
Python version
Please note that, from now on, we only provide support for the Python 3 version of TOM. TOM 0.2.2 for Python 3 is available on PyPI (https://pypi.python.org/pypi/tom_lib).
Adrien


fix(README): add note to remove throw statements else error

When I went to use this repo I accidentally forgot to remove the throw statements before the homeDirectory and username causing an error. In an effort to make sure someone else doesnt find the same fate I wanted to add a note outlining this and what you will see if you dont remove them.

After reviewing, it looks like you already added a note for this. So this PR should probably be closed


coprv2: hot-reloading of plugins

What problem does this PR solve?
Implements hot-reloading for coprocessor plugins, similar to the PoC.
Also adds some more public methods for PluginRegistry to deal with loaded plugins, e.g. the ability to unload plugins.
Again, I added some questions in TODO comments.
Issue Number: #9747
Problem Summary:
The coprocessor plugin framework should watch a "plugin directory" and automatically load/unload plugins that are created/deleted in the directory.
What is changed and how it works?
Proposal: RFC
What's Changed:
Related changes

based on PR #9912

Check List
Tests

Unit test

Release note

No release notes

/cc @andylokandy @skyzh

Not sure why, but the testmetrics::threads_linux::tests::test_thread_io_statistics is failing.

Not sure why, but the test metrics::threads_linux::tests::test_thread_io_statistics is failing.

That could be a temporary test failure. We may re-run the test later.

/tests

/test

/merge


Accessing block adding menu via keyboard

Current plan for adding a block: Press A key. This should allow you to tab through menu categories in addition to enabling number keys for category selection. When you select a category, all other categories are no longer accessible. Escape key can go back to category selection. Blocks can now be added to the scene using Tab + Enter or by hitting the relevant letter (a is the first block, b is the second block, and so on). When a block is added to the scene, the menu is automatically exited.

1-9 can be used to quickly jump to categories however, using letters to quickly choose a block would likely be more confusing than helpful to the user.


Move work items top and bottom for order of execution  (#558)

This PR will add the UI behaviour only, for moving a work item to either the top or bottom of the visible screen.
UI Task - #558

Current coverage is 60.61% (diff: 53.33%)

Merging #609 into master will decrease coverage by 0.47%

@@             master       #609   diff @@
==========================================
  Files            33         33
  Lines          1704       1719    +15
  Methods         345        349     +4
  Messages          0          0
  Branches        339        340     +1
==========================================
+ Hits           1041       1042     +1
- Misses          663        677    +14
  Partials          0          0

Powered by Codecov. Last update ca69680...3b18167

@nimishamukherjee will this fix also focus the list on the moved item? @almighty/uxd can you confirm the desired behavior, please?

@joshuawilson / @almighty/uxd , current behaviour:
when we click on the kebab menu, the work item gets selected and remains selected after being moved to the top or bottom.

Select a work item to move

Move the work item to top

Move the work item to the bottom

@almighty/uxd can you confirm that 'move to top' and 'move to bottom' is only moving to top and bottom of the viewable area?

@nimishamukherjee @almighty/uxd If the row is being displayed as selected, then the Detail View should be shown. After the row has been moved, it should fade back to de-selected.

@mindreeper2420 what about my question of "what is the bottom"? Bottom of the viewport or bottom of the list?

@joshuawilson the bottom is the bottom of the loaded list.

@mindreeper2420 ok, if we have a total list of 100 and we are at the top of the list, we have loaded 20 items. If I tell the item at position 5 to move to the bottom, does it go to 20 or 100? What you said is the loaded list and that is 20. I would expect it to go to 100.
I have a similar expectation that if I'm on 72 and I tell it to move to the top it should 1.

@mindreeper2420 - "If the row is being displayed as selected, then the Detail View should be shown. After the row has been moved, it should fade back to de-selected."
Few questions around this:

Does selecting the check-box imply selection of work item + opening of detailed view? Would the reverse hold good - deselecting the check-box would deselect the work item and close the detailed view?

Does selecting a work item for viewing the details select the check-box and display the top move button?

If the user wants to move a work item up by 3 rows, he would have to select the work item and click on "move up" three times. In this scenario, how would the detail view behave, if it was opened initially?

@nimishamukherjee

Correct - selecting the checkbox does not open the detail view.

Correct - the move dropdown is not enabled, as nothing has been selected to move. The move functionality in the kabob is neither enabled or disabled, though, so that is just there, but hidden behind the detail panel.

If the detail view is open at the time of selection and movement, then it would remain open until closed by the user. Moving items in the list does not force the detail panel to close.

@mindreeper2420 need your input on @joshuawilson 'squestion:

@mindreeper2420 ok, if we have a total list of 100 and we are at the top of the list, we have loaded 20 items. If I tell the item at position 5 to move to the bottom, does it go to 20 or 100? What you said is the loaded list and that is 20. I would expect it to go to 100. I have a similar expectation that if I'm on 72 and I tell it to move to the top it should 1.

@nimishamukherjee @joshuawilson a question I have before this is answered: is there a mechanism that informs the user there are more items that what have been loaded? In a standard paginated list, that would be easy with the pagination controls in PF. With the single-page list, there is normally a 20 of 100, etc. Is the UI showing that information?
If yes, then sending the item to the bottom makes sense to go the number 100. If not, then sending it to 100 when only 20 are visible (and perhaps known) would make it confusing and frustrating.

@mindreeper2420 there is support for display "20 of 100 items loaded", we just didn't have a requirement nor a design for it yet. I think we should add the "of total" and make move to bottom be the actual end of the list.

@joshuawilson agreed - I think that makes the most sense.

This was an an early design and when the implementation went to pagination
and then back to infinite scroll, it go lost in the shuffle.
The number is located here in the Visual Designs.
https://redhat.invisionapp.com/share/NU8YPHLJP#/195933683_A-0100
It needs to be tweaked to fall into place with our current layouts. It
would be great if Brian and Nimishia could connect on this.
Thanks All
-Monica
On Fri, Jan 6, 2017 at 9:28 AM, Adam Jolicoeur<EMAIL_ADDRESS>wrote:

@joshuawilson https://github.com/joshuawilson agreed - I think that
makes the most sense.
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fabric8io/fabric8-planner/pull/609#issuecomment-270913508,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARTwq_MAyLZDWxRIZDx2MZq71pDb5or1ks5rPk-YgaJpZM4LY6Z5
.

[test]

@nimishamukherjee When I move WI to top or bottom and then click on that WI then It's open wrong detail page.
It is working fine for the first time.

Thanks @vikram-raj good catch


“Unexpected response from browserscope.org” shouldn’t prevent new test creation

Whenever the “Unexpected response from browserscope.org” occurs, jsPerf should still create the test, just without a Browserscope ID.
When the test page is accessed we can re-attempt to get a Browserscope ID for this test.

+1 getting this error today

Also getting this error today

@maxbeatty’s fix is now live. @boozedog @OliverJAsh Everything ok now?

Great, thanks. I can now create a test, but can't edit it. When I try to
make a simple change I see "Not all tests inserted"
On Thu, 29 Dec 2016 at 21:04 Mathias Bynens<EMAIL_ADDRESS>wrote:

@maxbeatty https://github.com/maxbeatty’s fix is now live. @boozedog
https://github.com/boozedog @OliverJAsh https://github.com/OliverJAsh
Everything ok now?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jsperf/jsperf.com/issues/257#issuecomment-269693308,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4QCTIUedQoyjetFGCK7wRtMzr-R413ks5rNCBmgaJpZM4Km9qy
.

I'm seeing same as @OliverJAsh ... "Not all tests inserted"


Achievements can be covered up by the "Go back" button

it depends on the screen size

Thanks for this report. I have noticed that it has been getting close to touching on some screens but haven’t seen it be covered up. Thanks!


docs(examples): add autocomplete recipe

Those examples shows how to use an external react autocomplete component with react-instantsearch with two use cases:

targetting multiple indices
displaying both hits and facet values.

Those examples are boilerplate but will serve to derive the right API to perform the same thing.

One thing that could be done to lower the amount of code needed for this autocomplete is to enrich our searchState object with both hits and facet values results.

One thing that could be done to lower the amount of code needed for this autocomplete is to enrich our searchState object with both hits and facet values results.

This is something we will have to discuss soon. I am unsure if that's a good idea but it would simplify things clearly

Can you add it to the guide? Like "Autocomplete menu" guide.

Guide added.

Hi!
Are you able to link to the guide referenced above? My Google searching has only found the non-react autocomplete guide :s
Thanks in advance!

Hi @HemalR you can find the guide here: https://community.algolia.com/instantsearch.js/react/guide/Autocomplete_menu.html

You replied as quick as Algolia searches! Thank you :)


[Block] Elegant solution using a property

this will become more important since were using the hub for all things that are a state
sync: false
sync: 'downstream'
 sync: 'upstream'
sync: function (data, event, upstream) {}
sync: true
--> defaults to true
this way we can remove block
add exclusion for certain upstreams

upstream themselves can have an extra 'block' proeprty


Test cases for HHH-9343

@andrei-ivanov This is/was my first test case so I just copied the others from the class as an example and neither of them do. I'll be happy to add it, but if so should be it added to the others?

thanks @codylerum,
i have integrate the 2 tests and solved the issue


no amd support ?

In no place on your readme on github page you mention not supporting amd gpu's on windows,  always gpu & cpu yet I install to see there is no support , at least warn people ....

Thanks for pointing it out, updated the readme.


CLOUDSTACK-10295 Marvin: add support for password-enabled templates

Currently when new VM created, password property always set to 'password' even if it was auto-generated. Because of this impossible establish SSH connection to host during testing.
https://issues.apache.org/jira/browse/CLOUDSTACK-10295

good find @dmytro-shevchenko
@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1723

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-2266)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 22010 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2457-t2266-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_deploy_virtio_scsi_vm.py
Intermitten failure detected: /marvin/tests/smoke/test_ssvm.py
Intermitten failure detected: /marvin/tests/smoke/test_host_maintenance.py
Intermitten failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 65 look OK, 2 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_02_cancel_host_maintenace_with_migration_jobs
Error
2.37
test_host_maintenance.py

test_hostha_enable_ha_when_host_in_maintenance
Error
1.72
test_hostha_kvm.py

@dmytro-shevchenko ping

Here is result from our test. As testdata['templates']['builtin'] we using password enabled Ubuntu 14.04.
Test code:
from marvin.cloudstackTestCase import cloudstackTestCase
from marvin.lib.common import get_zone, get_domain, get_template
from marvin.lib.base import ServiceOffering, Account, VirtualMachine
from marvin.lib.utils import cleanup_resources

class TestVMConnectivity(cloudstackTestCase):

    @classmethod
    def setUpClass(cls):
        cls.testdata = cls.clstestclient.getParsedTestDataConfig()
        cls.api_client = cls.clstestclient.getApiClient()
        cls.domain = get_domain(cls.api_client)
        cls.zone = get_zone(cls.api_client, cls.clstestclient.getZoneForTests())
        cls.template = get_template(cls.api_client, zone_id=cls.zone.id,
                                    template_name=cls.testdata['templates']['builtin'])
        cls.service_offering = ServiceOffering.list(cls.api_client, name=cls.testdata['offerings']['compute'])[0]
        cls.account = Account.create(cls.api_client, cls.testdata["account"], domainid=cls.domain.id)
        cls.cleanup = [cls.account]

    @classmethod
    def tearDownClass(cls):
        cleanup_resources(cls.api_client, cls.cleanup)

    def test_vm_connectivity(self):
        self.debug('Creating new VM: {}'.format(self.testdata["virtual_machine"]))
        virtual_machine = VirtualMachine.create(self.api_client, self.testdata["virtual_machine"],
                                                accountid=self.account.name, domainid=self.account.domainid,
                                                serviceofferingid=self.service_offering.id,
                                                templateid=self.template.id, zoneid=self.zone.id,
                                                mode=self.zone.networktype)
        self.assertTrue(virtual_machine.passwordenabled)
        self.debug('VM name: {}, SSH IP: {}, SSH password: {}'.format(virtual_machine.instancename,
                                                                      virtual_machine.ssh_ip,
                                                                      virtual_machine.password))
        self.debug('Trying to ping <IP_ADDRESS>')
        ssh = virtual_machine.get_ssh_client(ipaddress=virtual_machine.ssh_ip, retries=3)
        response = ssh.execute("ping -c 3 <IP_ADDRESS>")
        self.assertIn("0% packet loss", '\n'.join(response))
        self.debug('Ping response: {}'.format('\n'.join(response)))

Here is part of MarvinLogs/8ODF2N/runinfo.txt output:
2018-02-20 16:01:21,117 - DEBUG - ::::::::::::STARTED : TC: test_vm_connectivity :::::::::::
2018-02-20 16:01:21,117 - DEBUG - Creating new VM: {'username': 'root', 'displayname': 'AutoTest VM', 'ssh_port': 22, 'privateport': 22, 'protocol': 'TCP', 'password': 'password', 'publicport': 22}
2018-02-20 16:02:31,567 - DEBUG - VM name: i-30-20-VM, SSH IP: 185.39.*.*, SSH password: J8gkqU
2018-02-20 16:02:31,567 - DEBUG - Trying to ping <IP_ADDRESS>
2018-02-20 16:02:34,471 - DEBUG - Ping response: PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=60 time=0.397 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=60 time=0.491 ms
64 bytes from <IP_ADDRESS>: icmp_seq=3 ttl=60 time=0.554 ms
--- <IP_ADDRESS> ping statistics ---
3 packets transmitted, 3 received, 0% packet loss, time 1999ms
rtt min/avg/max/mdev = 0.397/0.480/0.554/0.069 ms
2018-02-20 16:02:34,474 - DEBUG - TestCaseName: test_vm_connectivity; Time Taken: 73 Seconds; StartTime: Tue Feb 20 16:01:21 2018; EndTime: Tue Feb 20 16:02:34 2018; Result: SUCCESS

The tests that failed do not seem to be related to changes introduced here. So, the PR has integration tests, Marvin and reviews. I will merge it then.


Version flag in new command

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

The option --version shows in the ng new help section. It does not appear to do anything when creating a new project. This option may be from the version command.
Versions.
Angular CLI: 1.5.0
Node: 8.9.1

Repro steps.
ng new --help

The log given by the failure.
ng new [name] <options...>
    --dry-run Run through without making any changes. Will list all files that would have been created when running "ng new".
...
    --version (string)
...

Desired functionality.
Not sure

This is a flag accepted by the new application schematic to specify which version of the CLI for the new project to use.
It is not a value you need to set as the CLI automatically handles this for you.


Investigate GET /shops/:shop_id/orders leak

Serializer wants to retrieve all objects from the database, even if they're all already included.
Started GET "/shops/2/orders/" for <IP_ADDRESS> at 2016-03-22 20:19:34 -0400
Processing by OrdersController#index as HTML
  Parameters: {"shop_id"=>"2"}
  Shop Load (0.1ms)  SELECT  "shops".* FROM "shops" WHERE "shops"."id" = $1 LIMIT $2  [["id", 2], ["LIMIT", 1]]
  Order Load (0.7ms)  SELECT "orders".* FROM "orders" INNER JOIN "order_products" ON "order_products"."order_id" = "orders"."id" INNER JOIN "products" ON "products"."id" = "order_products"."product_id" WHERE "products"."shop_id" = $1  [["shop_id", 2]]
  Account Load (0.2ms)  SELECT "accounts".* FROM "accounts" WHERE "accounts"."id" IN (3, 4, 5, 7, 10, 12, 14, 16, 17, 19, 20)
  OrderProduct Load (0.4ms)  SELECT "order_products".* FROM "order_products" WHERE "order_products"."order_id" IN (14, 15, 20, 21, 22, 25, 29, 32, 42, 43, 44, 45, 46, 47, 48, 60, 67, 68, 69, 70, 71, 72, 74, 75, 76, 88, 90, 91, 93, 94, 96, 97, 98)
  Product Load (0.2ms)  SELECT "products".* FROM "products" WHERE "products"."id" IN (6, 5, 8, 9, 7)
  Shop Load (0.1ms)  SELECT "shops".* FROM "shops" WHERE "shops"."id" = 2
[active_model_serializers]   Account Load (0.1ms)  SELECT  "accounts".* FROM "accounts" WHERE "accounts"."id" = $1 LIMIT $2  [["id", 3], ["LIMIT", 1]]
[active_model_serializers]   OrderProduct Load (0.1ms)  SELECT "order_products".* FROM "order_products" WHERE "order_products"."order_id" = $1  [["order_id", 14]]
[active_model_serializers]   Product Load (0.1ms)  SELECT  "products".* FROM "products" WHERE "products"."id" = $1 LIMIT $2  [["id", 6], ["LIMIT", 1]]
[active_model_serializers]   Shop Load (0.1ms)  SELECT  "shops".* FROM "shops" WHERE "shops"."id" = $1 LIMIT $2  [["id", 2], ["LIMIT", 1]]
[active_model_serializers]   Product Load (0.1ms)  SELECT  "products".* FROM "products" WHERE "products"."id" = $1 LIMIT $2  [["id", 5], ["LIMIT", 1]]
[active_model_serializers]   CACHE (0.0ms)  SELECT  "shops".* FROM "shops" WHERE "shops"."id" = $1 LIMIT $2  [["id", 2], ["LIMIT", 1]]
[active_model_serializers]   Product Load (0.1ms)  SELECT  "products".* FROM "products" WHERE "products"."id" = $1 LIMIT $2  [["id", 8], ["LIMIT", 1]]
[active_model_serializers]   CACHE (0.0ms)  SELECT  "shops".* FROM "shops" WHERE "shops"."id" = $1 LIMIT $2  [["id", 2], ["LIMIT", 1]]
[active_model_serializers]   CACHE (0.0ms)  SELECT  "accounts".* FROM "accounts" WHERE "accounts"."id" = $1 LIMIT $2  [["id", 3], ["LIMIT", 1]]
[active_model_serializers]   CACHE (0.0ms)  SELECT "order_products".* FROM "order_products" WHERE "order_products"."order_id" = $1  [["order_id", 14]]
[active_model_serializers]   CACHE (0.0ms)  SELECT  "products".* FROM "products" WHERE "products"."id" = $1 LIMIT $2  [["id", 6], ["LIMIT", 1]]
[active_model_serializers]   CACHE (0.0ms)  SELECT  "shops".* FROM "shops" WHERE "shops"."id" = $1 LIMIT $2  [["id", 2], ["LIMIT", 1]]
[active_model_serializers]   CACHE (0.0ms)  SELECT  "products".* FROM "products" WHERE "products"."id" = $1 LIMIT $2  [["id", 5], ["LIMIT", 1]]
(continued for a long time, most of them cache hits)

Adding orders to shop model in issue #12 fixed this issue.


wormhole error

trying desperatly to bridge eth to solana using metamask and phantom, keeps saying
lock assets
waiting for metamask transfer approval
transfer failed.
tried removing history and history of cookies in chrome tried restarting locking and unlocking and nothing please help this problem is constant and persistant.

Please provide screenshots


fix: legacy card spacing and feat: legacy page mobile view text

@CIPHERTron
Please review this pr.


New functions & validation function

Added new validate files under perforce root function that raises error if files not under root
Removed default list arg for folder sync function
Added a delete changelist function
Added a revert changelist function (gets all files in a CL and reverts them all)

Fixed error in checking file exists. What does this mean though

"We shouldn't check if the file path is local before wrapping it in a P4File"

Can we get this package bumped on pip btw when it gets merged next


Added Pagination Config Option Through Headers.

This will allow for pagination information to be past through the
headers on both the request and response. The request should have a
Range-Unit: items and a Content-Range: 0-4. The 0-4 is the amount of
items per page via the current page.
In addition the response will return the Content-Range: 0-4/10, the 0-4
is the range you requested and the 10 will be the total posts.
This change pairs well with the
angular-paginate-anything library found here:
https://github.com/begriffs/angular-paginate-anything

This new configuration option probably should also be noted in an update to the project's README.

Ready to be reviewed.

Looks good :+1:


feature request: show only the path as the buffer's name

Did you check existing requests?

[X] I have searched the existing issues

Describe the feature
As a dirbuf/netrw long user, I love that their buffer name is only the name of the folder we are in.
Is that even possible to do on oil.nvim? Is it possible to remove the oil:// prefix from the buffer name?
I often use a particular use case. I go to the folder and want to :vimgrep the files under that folder. If the buffer name is only the path, I could do :vimgrep <pattern> <C-r>% to fill with the folder path.
Anyway, thanks for this plugin. You addressed all the problems I had on dirbuf.
Provide background
No response
Additional details
No response

The API includes the function get_current_dir which can be used to do things like this. I've also just added an action (actions.open_cmdline_dir) you can use to prepopulate the command line with the current directory as an argument. There's another action already to prepopulate the command line with the entry under the cursor (see :help oil-actions for a comprehensive list)

That's totally fair, the interface is different and does require remembering a mapping. Me not being able to remember my mappings was part of the reason I made this plugin in the first place :P
At the moment I'm pretty tied to the idea of oil buffers having a specific url prefix. It provides some nice guarantees that make the code less buggy and easier to maintain. I may at some point remove that restriction to enable full drop-in replacement for netrw, but I don't expect to do that anytime soon.
As another potential workaround, you could try using an abbreviation to trim the leading oil:// off of the buffer name:
vim.cmd.cabbr({ args = { "<expr>", "%", "&filetype == 'oil' ? bufname('%')[6:] : '%'" } })

Still not quite the same because instead of using <c-r> to get the contents of a register you're just typing the % directly, but it's at least closer to your workflow than the keymap.


error in chrome dev tools - Unchecked runtime.lastError: The message port closed before a response was received.

An error appears in the console in chrome dev tools upon launching Live Server
Unchecked runtime.lastError: The message port closed before a response was received.

browser - Chrome (desktop) version 72
Live Server: 1.30.2
Platform:  Windows
Visual Studio Code: latest

5.3.1

Please download 5.4.0 from this repo (under Releases), and try installing that manually.
View -> Command Pallete... -> Extensions Install from VSIX...

If there is any error, please paste the errors or screenshot from the Developer Tools.

Hello,
I removed the live server version I had and I downloaded LiveServer-5.4.0.vsix from releases and installed it, however it looks identical to what I had before (still says latest update was 5.3.1 in the description) and I still get the same error in console. Just to make it clear the extension works, but there's that annoying console error message still:
Unchecked runtime.lastError: The message port closed before a response was received.

Live server just updated to actual v 5.4 this time but the same error still persists in chrome.

Hi vk011 - This might be an issue with an extension you have from chrome. Try disabling them (chrome://extensions) and enabling them one by one to find the culprit. This worked for me.

@dchapo That was it man, thanks!

@vk011 Hey which extension caused the problem for you?

@rohitkrishna094  I think it was some kind of a video downloader, not sure which one though.

I had the same problem today and resolved it by disabling the following Chrome Extension;
https://chrome.google.com/webstore/detail/video-downloader-professi/elicpjhcidhpjomhibiffojpinpmmpil?hl=en

I had the same problem today and resolved it by disabling the following Chrome Extension;
https://chrome.google.com/webstore/detail/video-downloader-professi/elicpjhcidhpjomhibiffojpinpmmpil?hl=en

Hi @everyone. I figured out why it's happening. a small bug is there. You'll get an update with the fixes in this weekend. :)

Anything else? I still get such error with chrome 72.0.3626.121

For those coming here to debug this error in Chrome 73, it is because Chrome 73 onwards disallows cross-origin requests in content scripts.
More reading:

https://www.chromestatus.com/feature/5629709824032768
https://www.chromium.org/Home/chromium-security/extension-content-script-fetches

This affects many Chrome extension authors, who now need to scramble to fix the extensions because Chrome thinks "Our data shows that most extensions will not be affected by this change."

Hi, any updates on this?

Wondering if there is any update on this as well

Subscribing here.

I have solved this problem, some extention will cause this error, so you can use the no trace mode of chrome to develop, and ctrl + shift + N can come in this mode, then you can see the error is gone.

I have solved this problem, a video down-loader extension will cause this error,

https://chrome.google.com/webstore/detail/meddlemonkey/moihledlmchhofenpacbhphnbnpakgmo

was the Stylish extension for me causing this problem. I disabled and re-enabled it and now I don't get those logs

I had same Error with this following chrome Extension
https://chrome.google.com/webstore/detail/foxit-pdf-creator/cifnddnffldieaamihfkhkdgnbhfmaci

Unchecked runtime.lastError: The message port closed before a response was received.
Seems that some chrome extension that you have installed recently is creating problem. If you remove the recent installed extension then it will solve your problem.
See your chrome extension:
chrome://extensions

Hey i found to solutions to this bug/error:
Incognito browsing is quickest option or:
try disable extensions, and find which one cause that issue, for me was : Google Docs Offline
1.7

I have solved this problem, a video down-loader extension will cause this error,

https://chrome.google.com/webstore/detail/meddlemonkey/moihledlmchhofenpacbhphnbnpakgmo

solved,,,, thank you

I"m under the impression we're mixing two distinct cases in this discussion:
Case #_1. You're a user that have extensions installed, and one or more of them is causing the problem.

In this case, identifying and disabling the extension is the best way, as already suggested above.

Case #_2 - You're a developer and your extension is throwing the error.

Of course, you don't want to disable your own extension, but solve the problem. Despite the solution given by @ernsheong at https://github.com/ritwickdey/vscode-live-server/issues/270#issuecomment-473667533 (above) didn't work for him, I believe it's the best start point for it.
Also, I've been seen some solutions that recommend the developer to add return true to the the callback function (eg. https://github.com/mozilla/webextension-polyfill/issues/130#issuecomment-484772327) to make sure that the function will wait for the response.

My case is the #2 above and I'm still struggling to implement a complete solution. I'll publish the solution here, in case I find one.

I also solve this error. This error comes by what font extension see after disabling it.  optional follow me @atulcodex

Hi vk011 - This might be an issue with an extension you have from chrome. Try disabling them (chrome://extensions) and enabling them one by one to find the culprit. This worked for me.

Tanks!!
extension that was wrong here:
https://chrome.google.com/webstore/detail/cuponomia-cupom-e-dinheir/gidejehfgombmkfflghejpncblgfkagj?hl=pt-BR

Google Publisher Toolbar gave me this problem

Norton Safe Web <IP_ADDRESS> (Chrome Extension) was causing the error for me.

When I disabled 360 Total Security anti virus chrome extension, the problem was resolved.

Go to chrome://extensions/ and determine which extension is causing the error.

I too had this problem, it's resolved now! Just try disabling the chrome extension video down loader.  The same will work with
brave browser.

I haven't read the whole thread so don't know if it's already been mentioned, but my error was caused by Color Contrast Analyzer. Just thought I'd share!

I'm getting this error from the Shortkeys extension: Version-2.2.7

Hi vk011 - This might be an issue with an extension you have from chrome. Try disabling them (chrome://extensions) and enabling them one by one to find the culprit. This worked for me.

thanks, man I find my bug

Hi vk011 - This might be an issue with an extension you have from chrome. Try disabling them (chrome://extensions) and enabling them one by one to find the culprit. This worked for me.

In my case, the Pinterest Tag Helper extension was causing this error.
Thank you!

The same error in chrome was caused by the MeddleMonkey extension.

faced same issue on chrome, i disabled meddle monkey extension, and now everything is just fine.

faced same issue on chrome, i disabled meddle monkey extension, and now everything is just fine.

for me the culprit is color contrast analyzer extension :)

Solved the issue of the LiveServer Conflict, by deleting Typio Form Recovery

malwarebytes browser guard extension started showing these recently:

disabling the extension stopped them from appearing.
hiding them is also an option


Missing Requirements in Requirements.txt

Hi, when installing the node, I had 2 missing dependencies: open-clip-torch and pytorch-lightning.
You might want to add these to requirements.txt

yeah， i will add it


feat: user-specific todos!

Why is this pull request needed?
If @eoaksnes has made a todo to feed his dog, it's weird that it shows up on my todo-list.
What does this pull request change?
Makes todos personal, and makes the api serve them depending on who's logged in.
It's worth discussing whether this should "go deeper" than just the use-cases. Maybe the repos should care, or maybe even the client?
Issues related to this change:
Closes #129

Is the delete use case updated to check that it has the user has permission to delete the todo item?

Is the delete use case updated to check that it has the user has permission to delete the todo item?

Now that you mention it, it might not! I'll have a look at this!

I solved the above issues, and decided to go for a 404-approach (following @netr0m's comment). Re-review @eoaksnes?


Docker remote api (1.24)

Steps to reproduce the issue:

Send a REST API request to pull a non exist image from a private registry

Describe the results you received:
200 OK and raw response:
'''{"status":"Pulling repository nexus.example.com:5000/some_image"}
{"errorDetail":{"message":"Error: image some_image:1.0.0 not found"},"error":"Error: some_image:1.0.0 not found"}'''
Describe the results you expected:
HTTP 404 response

Can you provide more details?
The full issue template should be filled in.
At the moment I'm not 100% sure what the issue is.

@cpuguy83 Where can I find the template ?
Our problem is that docker remote api always response with status code 200 to requests
we expect it to response with other response codes to specific situations - in example:
if I trying to pull an image with docker remote api and the image can't be found in the repository it would be great it it can response with status code 404 (not found) and not 200.
This will make our life much easier.
Thanks

Description

Steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):
Output of docker version:
(paste your output here)

Output of docker info:
(paste your output here)

Additional environment details (AWS, VirtualBox, physical, etc.):

Closing because we didn't receive the requested information


BottomBar with DrawerMenu

I am using drawermenu with navigationview along with bottombar navigation, when I select an item from drawermenu, I need to remove the selected item from bottombar, is there any function for this?

Take a look at issue #324
Solved with pull request #370 , but it's not merge yet


Add SIGTERM support

Added support for catching the SIGTERM signal.
All Apple OS's send SIGTERM (like any good unix based system) when they want to request a graceful termination of an application, when the app doesn't quit in those circumstances, it'll receive a SIGKILL. This often gives us the opportunity to get a stack trace where we know Apple wants the app to be terminated but don't know why (watchdog events). This is just one more piece of information that can be added to the puzzle of figuring out unexplained app terminations.
Example in the Firebase Console:

Hi @naftaly, thanks for the PR! There is one CI failure: https://github.com/firebase/firebase-ios-sdk/actions/runs/8913792847/job/24481253930?pr=12881
This can be fixed by running the style script.


NearDrop not appearing in share menu on Mac

I tried sharing files from the finder, but NearDrop isn't appearing in the share menu on Mac. I'm nether getting the option to add it in the menu. I'm on MacOS Sonoma, if that matters.
PFA the screenshots:
Share menu:

Edit extension page:

Exact Same behavior on Ventura 13.4.1.
@grishka Is there some suggested way to figure out why this is not appearing? I tried to build mine using xcode and that too has same issue (on another thread you mentioned you got similar problems initially but later was resolved though finder restart, tried that doesn't work for me, including system reboot)

@kingster can you share the thread that you are referring to?

go to  settings and search for extensions , there you will see nearDrop in sharing section and the enable it. After that you will see it in share section

You should enable NearDrop sharing extension in Settings > Privacy & Security > Extensions > Added extensions.

I am also not seeing it. I tried looking at the extensions setting above, but it doesn't show the NearDrop in the lost for adding. Not sure what I could be doing wrong.

@scolapasta that's how it looks on my machine

Yup, not in mine:

Could it be related to how I installed it? I just downloaded the zip and unzipped in a subfolder of Applications.

I am also not seeing it. Used brew as well for installtion

You have to exit the menu and reopen it manually. It's a macOS bug that it for some reason doesn't show 3rd party apps when opened via the "edit extensions" button.

Hi, I have another issue like this.
I can not share the files via neardrop on photos app.
There is no issue on finder. I can share via neardrop on Finder.
Is it normal I can not use neardrop on Photos ??

Same issue -- not seeing it.
Also installed via brew

Just installed the newest release 2.0.2 and the extension is not present on the settings menu of my MAC. I'm running Sonoma.


Setting same mode twice resets the fan speed

Vallox seems to reset the fan speed to default, if status flag is set to same as it was.
e.g. fan_only to fan_only without any other variables being set.
Can be reproduced easily when using scenes from Home Assistant.

Fixed


Add note about support for specific values

closes #2713

Thank you and sorry for the disappointment!

Thanks for fixing my typo!


GWLB support for Transit Gateway.

GWLB should support Transit Gateway as a target.

hi, what if I want to add an Application loadbalancer behind spoke1 application, how can this be achived ? what are the chnages on routetable level?

Hello, I would suggest to read VPC Routing Enhancements and GWLB Deployment Patterns and Design your firewall deployment for Internet ingress traffic flows. I hope this helps.


SQL: Reset database zone configuration builtin

PR: https://github.com/cockroachdb/cockroach/pull/68280
From release notes:

Added a crdb_internal.reset_multi_region_zone_configs_for_database builtin to reset the zone configuration of a multi-region database. This builtin can be helpful in cases where the user has overridden the zone configuration for a given database and wishes to revert back to the original system-specified state. [#68280][#68280] {% comment %}doc{% endcomment %}

Captured in included docs.


TypeError: "cwd" must be a string on Node 8

After upgrading to Node 8, we have an error on startup when launching a NodeBB server.
This issue has been documented here : https://community.nodebb.org/topic/10765/error-starting-nodebb-after-upgrade
And here is the full error stack which has its origin in the daemon module used by NodeBB :
TypeError: "cwd" must be a string
    at normalizeSpawnArguments (child_process.js:380:11)
    at Object.exports.spawn (child_process.js:465:38)
    at Function.module.exports.daemon (/www/node-bb/node_modules/daemon/index.js:50:31)
    at module.exports (/www/node-bb/node_modules/daemon/index.js:25:20)
    at /www/node-bb/loader.js:223:21
    at FSReqWrap.oncomplete (fs.js:135:15)

According to the API documentation, process.cwd() is effectively a method : https://nodejs.org/api/process.html#process_process_cwd
The fix is quite trivial and i will provide a Pull Request for it.

Is there anyone who could review my PR ? It should be really fast i believe.

I noticed in the daemon source, when attempting to get the process.cwd, it's not calling the function, but using the function as the value.
index.js:
var cwd = opt.cwd || process.cwd

should be
var cwd = opt.cwd || process.cwd()

For anyone else affected, a quick workaround is to apply zipang's PR:
npm install 'github:zipang/daemon.node#48d0977c26fb3a6a44ae99aae3471b9d5a761085'

Paging @indexzero, is it possible for you to merge this PR?  It's a trivial bug fix, and would help us out.  Node 8 is now in LTS, so many more people will be using this.  Two character fix:
https://github.com/zipang/daemon.node/commit/48d0977c26fb3a6a44ae99aae3471b9d5a761085
Before:
var cwd = opt.cwd || process.cwd;

After:
var cwd = opt.cwd || process.cwd();

Thanks!

@indexzero Ping, can we get #42 into an NPM release?

The workaround for this issue should be to pass in cwd in options, which typically would be set to /.

same issue here but when implemented in modern node way it works

@stern0 What is the "modern node way"?

@jhuckaby actually I was wrong the same issue persists

@jhuckaby but the latest PR works

@stern0 Yup, just need OP to merge it ;)

Thank you @silverwind.  If you don't did it, I'll fork daemon project to fix this issue. Congratulations


build: support building inside a container using stacker

Thank you :)

Codecov Report

Merging #12 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #12   +/-   ##
=======================================
  Coverage   43.07%   43.07%
=======================================
  Files          10       10
  Lines        1170     1170
=======================================
  Hits          504      504
  Misses        587      587
  Partials       79       79

Continue to review full report at Codecov.

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


add aws eks hello world example

Thank you. What is new vs. https://github.com/VirtusLab/besom/blob/main/examples/aws-eks/Main.scala ?

Thank you. What is new vs. https://github.com/VirtusLab/besom/blob/main/examples/aws-eks/Main.scala ?

In this example, I added the ability to see running application on a browser.

Could you incorporate this feature into existing example?


High Energy/CPU usage after reconnecting on macOS

🤷🏽♂ Current Behaviour
Similar to https://github.com/sidneys/pb-for-desktop/issues/25
The energy consumption is low after start.
After I select "Reconnect..." tray menu item or if the app reconnects automatically, one of the helper processes starts to use a lot of CPU/Energy (pretty much as in https://github.com/sidneys/pb-for-desktop/issues/25).
Logging out and then back in helps to resolve it until a next reconnection
🎯 Expected Behaviour
CPU/Energy consumption after a reconnection remains as low as it was before the reconnection.
👟 Steps to Reproduce (S2R)

Launch the app and log in
Select "Reconnect..." form the tray menu

🏡 Environmental Context
App Version
v4.3.0
Installation Type
Disk image
Operating System
masOS Sierra 10.12.3

@electronic-dk
Thanks for the report. Can you please test this again with the latest release? Some core components related to performance have been replaced. Thanks.
--> Link

@sidneys unfortunately, I'm still experiencing this issue.

I'll try to investigate more

@sidneys I'm not experiencing this issue in v5.0.0 anymore. Thanks so much for fixing it 👍

@electronic-dk Any updates on this? I couldn't reproduces this the releases after 5.3.0.

@electronic-dk
This is not reproducible anymore - if it reoccurs, please comment or create a new issue. Closing.


IList.Contains LINQ query does not work

The following doesn't work:
public class TestModelObject {
    public Guid Id { get; set; }
    public string Title { get; set; }
    public IList<string> Tags { get; set;}
}

using (var session = _documentStore.LightweightSession()) {
    var tag = $"tag-{Guid.NewGuid().ToString()}";

    var obj = new TestModelObject {
        Title = $"Object created at {DateTime.Now}",
        Tags = new List<string> { "abc", "xyzzy", tag }
    };

    session.Store(obj);
    session.SaveChanges();

    var objByTag = session
        .Query<TestModelObject>()
        .Where(tmo => tmo.Tags.Contains(tag))
        .FirstOrDefault();
}

objByTag is null. As long as Tags' type is either List<string> or string[], it works, but not when Tags is IList<string>. According to your documentation, this should be supported.
This is as of Marten 1.0.0-alpha-644 on .NET Core 1.0 on OS X 10.11.5.
(PS: Could you also add support for querying on IReadOnlyList<T> the same way as on IList<T>?)

@JesperTreetop I'll deal w/ IReadOnlyList shortly, but for right now, I'm not able to reproduce your problems with IList<string>. This test passes for me:
        public class DocWithLists
        {
            public Guid Id;

            public IList<string> Names = new List<string>();

        }

        [Fact]
        public void can_do_contains_against_IList()
        {
            var doc1 = new DocWithLists {Names = new List<string> {"Jeremy", "Josh", "Corey"} };
            var doc2 = new DocWithLists {Names = new List<string> {"Jeremy", "Lindsey", "Max"} };
            var doc3 = new DocWithLists {Names = new List<string> {"Jack", "Lindsey", "Max"} };

            using (var session = theStore.OpenSession())
            {
                session.Store(doc1, doc2, doc3);
                session.SaveChanges();

                var ids = session.Query<DocWithLists>().Where(x => x.Names.Contains("Jeremy")).Select(x => x.Id).ToList();

                ids.Count.ShouldBe(2);
                ids.ShouldContain(doc1.Id);
                ids.ShouldContain(doc2.Id);
            }
        }

The existing tests on IList are also passing.

@JesperTreetop IReadOnlyList's aren't going to happen without some customizations to Newtonsoft. I'm sorry, but that's a no-go for now.

Very strange, I will re-check and if I still get the same result I'll clone the repository and see if the test passes for me. Mine was a property and not a field but that probably does not make any difference.
Since IReadOnlyList<T> support has been in Json.NET since 5.0, what's missing that would require the customizations? Just curious.


[Transaction builder generator] Improve python code

Motivation
This follows up with https://github.com/facebookincubator/serde-reflection/pull/20 on improving type-checking for generated python.
Test Plan
Temporarily requires a local checkout of serde-reflection for pyre "typeshed".
cargo x test -p transaction-builder-generator --ignored

@bors-libra r=thefallentree

:pushpin: Commit a508f4c has been approved by thefallentree

@bors-libra r+

:bulb: This pull request was already approved, no need to approve it again.

There's another pull request that is currently being tested, blocking this pull request: #4722

:pushpin: Commit a508f4c has been approved by matbd

:hourglass: Testing commit a508f4cae87c8a60867c0c5532a956386bcb6e13 with merge 15b08482f64e47e57cf68360163303247c06128e...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: matbd
Pushing 15b08482f64e47e57cf68360163303247c06128e to master...


Autosuggest dropdown is not clickable

Browser
No response
Package version
3.0.7
React version
18.2.0
Description
I have the Autosuggest dropdown within a container. When the dropdown is opened, it is not clickable.  The options are only selected with the down arrow + enter.
Source code
return (
    <Container header={<Header variant="h2">Association</Header>}>
      <SpaceBetween direction="vertical" size="l">
        <NotificationBar error={errorMessage} clearNotifications={errorMessage === undefined} />
        <ColumnLayout columns={4}>
          <FormField label="Association ID" constraintText="10 character limit. No spaces">
            <Autosuggest
              enteredTextLabel={(value) => `Use: "${value}"`}
              options={associationCodeArray}
              onChange={({ detail }) => {
                setAssociationCode(detail.value);
              }}
              value={codeValue}
            ></Autosuggest>
          </FormField>
          <InputFormField
            label="Association Name"
            value={formData.associationName}
            editable={_editable}
            transformOptions={{ allCaps: true }}
            onChange={(_e, _error, transformValue) =>
              onInputChange('associationName', transformValue!)
            }
            validation={{ max: 50, required: true }}
          />
          <InputFormField
            label="Termination Date"
            constraintText="mm/dd/yyyy"
            value={formData.terminationDate.toString()}
            editable={_editable}
            onChange={({ detail }) => {
              onInputChange('terminationDate', detail.value);
            }}
          />
          <Box padding="xl">
            {isBusy ? (
              <Spinner />
            ) : (
              <BusyButton variant="primary" onClick={() => onAdd()}>
                Add
              </BusyButton>
            )}
          </Box>
        </ColumnLayout>

        <TemporaryEditableDataTable
          data={props.data}
          modelName={'Association'}
          onDelete={(d) => onDelete(d)}
          converters={converterMap}
        />
      </SpaceBetween>
    </Container>
  );

Reproduction
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems

Hi @yer421, could you share a reproducible example and/or a screenshot/video of the issue.
The code example code you've shared contains custom components and variables which are not referenced. Having a reproducible example makes investigating less like a guess-game.

Hello @yer421!
Please provide a reproducible example (ideally with CodeSandbox) so we can investigate this issue. Otherwise this issue will be resolved.

I've included a video of what happens when trying to click on an item of the dropdown of the Autosuggest tool.
The dropdown list comes from an API call so I can't include that within my code.  As you can see from the video, using the down arrows and then clicking Enter works, but the mouse click does not work in the component.
Let me know if you need anything else.
Thanks,
Yer
From: Timo Gasda @.>
Sent: Tuesday, January 3, 2023 5:04 AM
To: cloudscape-design/components @.>
Cc: Yer Lo @.>; Mention @.>
Subject: Re: [cloudscape-design/components] Autosuggest dropdown is not clickable (Issue #578)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
CAUTION: This email originated from outside of Navitus or Lumicera. Do not click on any links or open any attachments unless you recognize the sender and know the contents are safe. Questions? Contact the Helpdesk @ x4444
Hello @yer421https://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fyer421&data=05|01|yer.lo%40navitus.com|5c844c6a36c247494fc208daed7a4260|00dbd1fbc5ea4664b0486e809757474c|0|0|638083406627228521|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=SrcptuPItxndqucakgK7cGDA1z1oWRYcAIsURQwwDWM%3D&reserved=0!
Please provide a reproducible example (ideally with CodeSandboxhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fcodesandbox.io%2Fs%2Fcloudscape-design-system-react-javascript-ljs1t7%3Ffile%3D%2Fsrc%2FApp.js&data=05|01|yer.lo%40navitus.com|5c844c6a36c247494fc208daed7a4260|00dbd1fbc5ea4664b0486e809757474c|0|0|638083406627228521|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=0AGMZpm9HivHoNLboEAemISXFgoy4aFVa7gp7c%2BIuiM%3D&reserved=0) so we can investigate this issue. Otherwise this issue will be resolved.

Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcloudscape-design%2Fcomponents%2Fissues%2F578%23issuecomment-1369638542&data=05|01|yer.lo%40navitus.com|5c844c6a36c247494fc208daed7a4260|00dbd1fbc5ea4664b0486e809757474c|0|0|638083406627384770|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=By%2BPOsvJlLuQJjC%2BTzN7NPXDMs1dd9k%2Fyx33DwHzHmU%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPSST7GRPVUAKXRFYLCDPUTWQQBTDANCNFSM6AAAAAAS66FNCY&data=05|01|yer.lo%40navitus.com|5c844c6a36c247494fc208daed7a4260|00dbd1fbc5ea4664b0486e809757474c|0|0|638083406627384770|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=692Gs6LM84SV2Tl3JVheceag4L9f1GuiwqS2Dabyhjk%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

The information in this email message is intended only for the confidential use of the individual or entity to which it is addressed. It may contain information that is privileged, confidential, and exempt from disclosure under law. If the reader of this message is not the intended recipient, you are hereby notified that you have received this document in error and that any review, dissemination, distribution or copying of this message is strictly prohibited. If you have received this communication in error, please notify us immediately at the email address listed above. Thank you.

Hi @yer421, it sounds like you responded to this thread via email with an attachment, but GitHub did not include the attachment here. Could you try uploading it through GitHub's web interface?
Also, if you can reproduce the issue with code that you can share (e.g. by replacing the API call with placeholder data, removing unrelated components etc), it would be great if you could include the code (e.g. in CodeSandbox).

Closing due to inactivity. Feel free to reopen if you have more questions


tee should allow its iterators to return early

A selling point of tee the way we wrote it is that it will work on infinite iterables -- as long as the consumers keep consuming each iterable resuling from the tee all the old entries are removed from the dequeues and we don't keep much of anything in cache.
But what if one of those consumers stops consuming? Now its cache will begin to fill up -- essentially creating a memory leak.
We should return an iterable with a return method, which method cleans up by destroying that iterable's tee cache.

this has been fixed with the new fork


Cannot open Task grid in Teams

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

ID: f9b5cae0-20ce-25f9-12a1-ad1d504142e9
Version Independent ID: 558c0557-98cc-2070-c83c-759a50b9c533
Content: Troubleshoot working in the Task grid
Content Source: articles/project-management/projectplanningUIFAQ.md
Service: dynamics-365-project-operations
GitHub Login: @ruhercul
Microsoft Alias: ruhercul

@rajatk09 Thank you for the feedback.
@ruhercul Russel, can you take a look at this and update the topic if needed?


Interceptors?

Is there the concept of interceptors in fetch? I'd like to add a header to all of my requests automatically.

Yes, it's called service worker!

@domenic
Do you know if there is an available polyfill for this?

@domenic - can you provide an example of how to accomplish this with service workers?

@lynndylanhurley I'm sorry, I don't have the time to do so. You might want to read up on some articles on service workers, or ask a more specific question on a site like Stack Overflow.

Service workers don't seem to be supported at all in IE or Safari. (see also here)
Is there another way to automatically include a header with each request using this library?

Is there another way to include a header with each request using this library?

Yes:
var originalFetch = window.fetch
window.fetch = function(url, options) {
  if (!options.headers) options.headers = {}
  options.headers['x-custom'] = 'here'
  return originalFetch(url, options)
}

@mislav - I'm trying to create an authentication library that uses fetch. If the only way to add default request options is to redefine the method, my concern is that another library may take the same approach and overwrite the options from my library.

To be clear, I don't think redefining the native function is a good approach. However, I think that's the only option you have if you want to hook into every fetch() call on the page.
If you're building a library, I suggest that your library exposes its own method like authenticatedFetch() which takes the same options, does the logic you need to do, and returns the same promise that a fetch() returns.


The New Application / Health checks close button (x) does not work

JIRA Key: MARATHON-448
Reporter: Orlando Hohmeier
Priority: Medium
Description:
The New Application / Health checks close button (x) does not work and should be removed.

Try to add an additional health check. Then you can remove the health checks with this button otherwise it clears the last standing health check.

Not a bug to me, but a question of UX.

Not a bug.

Okay... It's a clear/remove button and it does work. We should use a different icon as it looks like the close button.

What I recommend

Change the "x" icon for health checks to a minus style icon "–" so it's not confused with the modal close action
Change the button copy to "Add Another Health Check"
If the user clicks the minus icon on the first health check, remove the health check fields but keep a button with "Add A Health Check"

p.s. a whole new "Create" experience is on the roadmap

@leemunroe: Perfect intermediate suggestions! Thanx!


Question to conf true\false is case insensitive?

Is true equal True while conf parsing?

True and False are case sensitive.
https://github.com/doitsujin/dxvk/blob/master/src/util/config/config.cpp#L576

The example file even kind of tells you that True is the one to use.
Also, tolower is locale sensitive, i.e. really bad.

.ToLower(CultureInfo.InvariantCulture)

We'll keep that in mind when we rewrite DXVK in C#...

setlocale

I don't really see how this is a big deal in the first place, it hasn't been for the past three years and given your behaviour in this thread i'm honestly inclined to leave it as-is out of spite.

Don't get me wrong - I wasted a lot of time because of this misunderstanding. I just couldn't imagine that it could be so bad.
English is not my native language. I didn't mean to offend anyone.
I am not a C++ programmer, and I hardly use it, but even for me there is no difficulty in this.
If you think the current implementation of config parsing is good, then let it stay as it is.
I don't think so, but that's my personal opinion. I would be fired from my job if I made such an implementation :)


Possible compiler backend issue in 0.4.0-SNAPSHOT .gt. M2

The presenting problem is that scalactic (3.1.x, dated approx 2019-07-28)  fails to
compile against SN 0.4.0-SNAPSHOT (dated 2019-07-29).
I have seen this twice; once in the middle of a debugging frenzy and just now
from a rested, cold, clean start.
The exact same scalactic  code compiles against SN 0.4.0-M2. Both the scalactic and
scalatest TestRunners complete successfully.
I suspect that this is a name-mangling or inline bug.
The relevant code in java.nio and scalanative/mumble/io/package.scala has not changed in over two years.  java.nio.ByteBuffer.clear() exists in the source file (and is probably heavily used by the SN compiler backend).
The error & top of the stack trace:
## Exception when compiling 83 sources to /mnt/QuincyQHdd1/lee49/DevoHdd1/scala_native/WorkSpace_1/debug_SN_ScalaTest_Failure_I1400_V3_SN-0.4.0-SNAPSHOT/scalactic-macro.native/target/scala-2.11/classes
[error] java.nio.ByteBuffer.clear()Ljava/nio/ByteBuffer;
[error] scala.scalanative.io.package$.withScratchBuffer(package.scala:15)
[error] scala.scalanative.nscplugin.NirGenFile$class.genIRFile(NirGenFile.scala:30)
[error] scala.scalanative.nscplugin.NirGenPhase.genIRFile(NirGenPhase.scala:12)
[error] scala.scalanative.nscplugin.NirGenPhase$NirCodePhase$$anonfun$apply$2.apply(NirGenPhase.scala:91)
[error] scala.scalanative.nscplugin.NirGenPhase$NirCodePhase$$anonfun$apply$2.apply(NirGenPhase.scala:89)
[error] scala.collection.parallel.mutable.ParArray$ParArrayIterator.foreach_quick(ParArray.scala:143)

I know of no way to cut this down to the ever valuable  small reproducer.

Oops!   I created an issue in the wrong Repository. I hope I did not
insult anybodies dog or code  Sorry for the noise.
I believe (strongly) that this is a Scala Native issue and had intended to post & track there.


fix: [#4746] Fix INodeBuffer in TypeScript 5.6 and ESNext target

Fixes #4746
Description
This PR fixes an issue with INodeBuffer incorrectly implementing the Uint8Array.

[!NOTE]
We are working on adding support for TS 5.6 and ESNext target in BotBuilder in a separate PR, which are impacting some of other libraries.

Specific Changes

Removed entries, keys, and values methods from the INodeBuffer, so they are inherited correctly across different TS versions.

Testing
The following image shows the botframework-streaming library building correctly with TS 5.6 and ESNext target.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 84.997%

Totals

Change from base Build<PHONE_NUMBER>2:
-0.4%

Covered Lines:
20351

Relevant Lines:
22910

💛  - Coveralls


Update docker hub links (from pegasyseng to consensys)

Pull request checklist
Before creating the pull request
Make sure that:

[x] you have read the contribution guidelines.
[x] you have tested your changes locally before submitting them to the community for review.

After creating your pull request and tests finished
Make sure that:

[ ] you have fixed all the issues raised by the tests, if any.
[ ] you have verified the rendering of your changes on
ReadTheDocs.org PR preview
and updated the testing link (see Testing).

Describe the change
Updated docker hub links from pegasyseng to consensys.
Issue fixed
fixes #127
Testing

Open https://docs.ethsigner.consensys.net/en/stable/HowTo/Get-Started/Use-Docker/
Check that we are referring to https://hub.docker.com/r/consensys/ethsigner

 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.

Merge this only after Ethsigner v21.10.0 has been released.


Minor fix in LOWER-BODY

Closes #118.
This PR fixes the following test case on Lua hosts:
(test= 'b ((fn () (case 2 0 (do) 1 'a 2 'b))))

(Without this PR, ((fn () (case 2 0 (do) 1 'a 2 'b))) incorrectly evaluates to nil on Lua.)

Actually, I think this is better fixed in lower-statement. One moment.

Ok, if you like this PR, it's ready to merge.


Stop posting endpoint from throwing invalid exceptions

The posting endpoint wrapper was throwing exceptions when json errors were present (not 4xx's/5xx's but a 200). Since we actually need to look at the errors stop this from happening.
I handled this by not having a default parseBody function but allowing the mobile code to handle the promise directly.
:fish: @schwers

💅  We talked about this in person, but just wanted to leave a note here. Things to try were testing if '.json' on the endpoint made error parsing easier, and seeing how throwing a BadCaptcha error here felt.

@schwers :haircut:

:fish:

:fish: 🐠


fix stm32f4 links to st website

Should I also update links in the README or they are automatically generated in some way?

I have updated the links and the README. :smile:


Initial inserts

Hi,
Is there any way in which inserts can be done on app installation?
Like we specify the inserts.sql in sugar_ugrades under assets folder.
I tried with version 1 and 1.sql didnot work.
Thanks.

Hello,
I have the same problem.
Thx

same problem...

Hi guys.
I made a workaround to solve that.
1- Create a 0.sql file in sugar_upgrades with your initials inserts;
2- Add this code in your main activity:
if (AppPreferences.getInstance().isFirstExecution()) {
    SchemaGenerator generator = new SchemaGenerator(this);
    generator.doUpgrade(new SugarDb(this).getDB(), -1, 0);
    AppPreferences.getInstance().setFirstExecution(false);
}

*AppPreferences is a controller to manipulate my SharedPreferences that has a boolean property indicating if the app is running for the first time.
I hope it will help somebody else :)

Hello guys! This trouble has been solved by PR 577 which adds a method called afterTableCreated which reads a folder in assets that is called sugar_inserts, so you can put a custom .sql script to populate your db with fresh data.
It only executes when schemaGenerator is called.
If you want to probe this new feature you should use SugarORM from master branch.
We are waiting for a patch release version of sugar.

Hello guys! This trouble has been solved by PR 577 which adds a method called afterTableCreated which reads a folder in assets that is called sugar_inserts, so you can put a custom .sql script to populate your db with fresh data.
It only executes when schemaGenerator is called.
If you want to probe this new feature you should use SugarORM from master branch.
We are waiting for a patch release version of sugar.

Hello guys! This trouble has been solved by PR 577 which adds a method called afterTableCreated which reads a folder in assets that is called sugar_inserts, so you can put a custom .sql script to populate your db with fresh data.
It only executes when schemaGenerator is called.
If you want to probe this new feature you should use SugarORM from master branch.
We are waiting for a patch release version of sugar.


feat: implementation / enhancements of analytics

This PR adds analytics to most parts of the app, as requested by the product team (@moiz994).
Note: @marcotuts We have discussed it internally, and I have renamed the already implemented analytics names for consistency between already implemented and newly implemented analytics, e.g  Forgot_password_Clicked to Logistration_Forgot_Password_Clicked, Cookie_Policy_Clicked to Profile_Cookie_Policy_Clicked.

@volodymyr-chekyrta The tests are passing locally but failing on the PR, and I'm not getting any useful insights from the logs. Could you please look into it and see what needs to be fixed.

LGTM perhaps needs to coordinate with @shafqat-muneer and add analytics for Calendar sync too (maybe after his PR merging?)

We have created a separate task for calendar integration events.


Allow install to be called with date object

Fixes #73

Not sure about the test however :(

Five breaking tests with half a line of diff, that is impressive :-D Seriously, you might look into setting a breakpoint in some of those tests to inspect what is going on. I cannot see it from just glancing, at least.

what can i say im good at breaking things. i'll take a look at it tomorrow morning.

@fatso83 so I've got a question, I've fixed the tests but I notice this test https://github.com/sinonjs/lolex/blob/78323dc3ff6b69c76dcc9a9b43a106c9859d3d7a/test/lolex-test.js#L1720-L1725 that tests the exact opposite of what I'm trying to achieve here? is that intended? or can i just drop that test?

Sorry about not answering that for so long, btw. Not sure, I'll try to git blame it to get some context this week (holidays).

totally forgot about this, and I have been way too busy (and is). hoping someone can look into this...

OK, I am terrible at letting things go, so I went down the rabbit hole of git blame, back to the time lolex was first extracted from Sinon, and then further back to when this was first introduced as part of a fix to sinon/sinonjs#49 . Please check out the conclusion on the discussion there.
If I understand the discussion correctly, it seems as if @cjohansen made the call that since the original Date() constructor is not supposed to accept a Date object, then neither should we. Not sure I am totally following that logic, as sinon.useFakeTimers could well accept a date and then extract the milliseconds out of that object. Not that you are doing that here, anyway, so ...
This requires a deeper look into the Lolex code to see how things are interconnected so we don't introduce a bug by allowing Date objects where there shouldn't be one.

This has been manually applied to master. Not sure why Github doesn't pick it up.


Unity hololens using holotoolkit FOV always 16.97196

I am using unity with holotoolkit for developing app for hololens. The issue is the Field of View (FOV) of the main camera is always 16.97196 no matter what values is input. I even added a script to deliberately set the FOV value to 60 but it resets to 16.97196. Can the FOV value be set the user requirements.

Isn't that because of the limited FoV from a hardware perspective?

Thanks for the reply. The reason why i want to change FOV because I need to play 360 video in a sphere. The video looks very close and not able to get the desired result. The standard FOV for 360 sphere is 60

@tipking, I don't think there's a way to change this, because it's hardware dependent.

As a side-note, scaling the sphere up shouldn't actually display more video content because the FOV angle is independent of the radius. It will, however, make it look further away if you are using a monoscopic video.
So if it looks close to your face then a larger radius will make things feel more natural, but if the issue is the amount of content you can see at once, that is bound to the hardware field of view.

This seems resolved. That value comes from the device, so changing it doesn't mean anything to the hardware.


Integrated terminal stopped to work for debugging after insider update

Version: 1.50.0-insider (user setup)
Commit: 9e505675670d65138405321a60b0df4ddec28799
Date: 2020-09-16T06:49:37.816Z (7 hrs ago)
Electron: 9.3.0
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042
Steps to Reproduce:

Open any project folder with debugger configured with "integratedTerminal"
Hit F5 to start debugging

Debug toolbar will appear and integrated terminal opens at bottom, at this moment get an error message:
The terminal process "C:\Program Files\PowerShell\7\pwsh.exe" terminated with exit code:<PHONE_NUMBER>.
After a research I made it work, but only with Command Prompt as default terminal and not the powershell 7.
Following this guide: https://code.visualstudio.com/docs/supporting/troubleshoot-terminal-launch#_terminal-exits-with-code-3221225786-or-similar
The error occurs only with powershell core (I'm with version 7.0.3).
Also tested with Windows Powershell, gets no error but doesn't work too.

So you did disable legacy console mode? You should also try this if you have anti-virus software https://code.visualstudio.com/docs/supporting/troubleshoot-terminal-launch#_error-connectnamedpipe-failed-windows-error-232
Do you have any terminal settings?

Yes, the normal cmd works with or without legacy. And no anti-virus, only default "Windows Defender".

Do you have any terminal settings?

That's the problem:
"terminal.integrated.shell.windows": "C:\\WINDOWS\\System32\\cmd.exe", << Works
"terminal.integrated.shell.windows": "C:\\WINDOWS\\System32\\WindowsPowerShell\\v1.0\\powershell.exe", << No error but doesn't work
"terminal.integrated.shell.windows": "C:\\Program Files\\PowerShell\\7\\pwsh.exe", << Gives<PHONE_NUMBER> error and doesn't work

The same setup worked before update to 1.50.0-insider.
Need to do some config into powershell 7 terminal?

@Tyriar
This error occurred on the latest Insider Code and 7.x Powershell.
I have encountered this error on three different computers, and it did not happen in previous versions of Insider Code.
Even though I turned on Legacy Console Mode on Powershell, the error persists.
I think this error is not directly related to Legacy Console Mode

@lucianotres @zheung you must turn off legacy console mode, the terminal will break if it's enabled when the new backend conpty is used (Windows 1909+).

@lucianotres @zheung you must turn off legacy console mode, the terminal will break if it's enabled when the new backend conpty is used (Windows 1909+).

It is OFF!
It is turned off by default. I never checked this option

Yep. I've also recently (last week) been affected by this issue. (I tried with PowerShell 7.1.0-preview.7 and the non-preview latest version.)
I've subsequently changed the terminal to cmd instead of Powershell to work around the issue.

Yep. I've also recently (last week) been affected by this issue. (I tried with PowerShell 7.1.0-preview.7 and the non-preview latest version.)
I've subsequently changed the terminal to cmd instead of Powershell to work around the issue.

Exactly, only cmd works.
Yes @Tyriar, legacy console mode it's off. The issue it's with powershell only.
I'm experiencing the same that @zheung showed at gif.

I am seeing this behaviour as well in debugging of Python programs. Works fine on VSCode, fails in the manner detailed above in VSCode-Insiders. Powershell core 7.0.3.

Same issue here.
Powershell core 7.0.3 with VSCode-Insiders update to the lastest version of 1.50.0 and then same error msg.
The terminal process "C:\Program Files\PowerShell\7\pwsh.exe" terminated with exit code:<PHONE_NUMBER>.

Same issue too.

FWIW I notice that the error code<PHONE_NUMBER>, 0xC000013A) apparently represents when an application is exited via ctrl+c (STATUS_CONTROL_C_EXIT):
https://docs.microsoft.com/en-us/openspecs/windows_protocols/ms-erref/596a1078-e883-4972-9bbc-49e60bebca55?redirectedfrom=MSDN

The error notification now features a link to troubleshooting docs.


Don't break on query string.

@tptee this allows routes to match when query params are included in the url string.

Coverage increased (+0.04%) to 97.059% when pulling d47ef7c8db81ccf4d007883297f9f3ec2dc65a29 on dont-break-on-query-string into 947552bcadc2ed537c593e87f6c7a2b20b1a0f3e on master.

Code LGTM apart from the checked in dist files.

Coverage increased (+0.04%) to 97.059% when pulling a28b74bbe5a51470419e1efc004f3529a8df6247 on dont-break-on-query-string into 947552bcadc2ed537c593e87f6c7a2b20b1a0f3e on master.

Coverage increased (+0.04%) to 97.059% when pulling a28b74bbe5a51470419e1efc004f3529a8df6247 on dont-break-on-query-string into 947552bcadc2ed537c593e87f6c7a2b20b1a0f3e on master.


Info on restructuring in changelog

Pending #577
Contributor Checklist:

[x] I have added myself to the creators.json file
[x] I have added a changelog entry for my contribution

This looks good. Do you want to have this merged as-is, or do you want to include more documentation changes related to the restructuring in this PR?

This looks good. Do you want to have this merged as-is, or do you want to include more documentation changes related to the restructuring in this PR?

Perhaps we can merge now to work step-by-step and keep the parts manageable.

Perhaps we can merge now to work step-by-step and keep the parts manageable.

:+1:


Updates team ownership

Adds submodules for PXC-Release project as part of new feature development
Directed to submit this PR as the new way to add members to projects

i followed the process mentioned above to be authorized uner the CLA - see Upstream Contrib Ticket 4169647
what else is needed on our end to get the this file update so we can be working on active customer stories? our team is completely blocked at this point

@rkoster @beyhan is this still relevant within the Foundational Infrastructure working group, especially now that we're close to clearing out all the legacy teams in the CF GitHub org?

Closing, as #387 has removed all the team structure not coming from the Working Group or TOC definitions. If the Integrated Databases area in the Foundational Infrastructure working group needs to own other repos or to add other members as approvers, please submit PRs to change the definition of the WG in https://github.com/cloudfoundry/community/blob/main/toc/working-groups/foundational-infrastructure.md.


Read-only file system (os error 30) on MacOS

I just tried ollama-grid-search for the first time. I'm using MacOS 14.4 and I'm using v0.1.0 of Ollama Grid Search.
For each permutation it takes a while, during which ollama keeps my GPU busy, and then displays:
Read-only file system (os error 30) before moving on to the next permutation. It doesn't display the results of the inferences, just the error messages.
I can't tell where it is trying to write, but on MacOS, the root of the filesystem (/) is read only

Nice tool!
Same here on Arch Linux using the .AppImage from a directory that contains all my AppImages.
Thank you,
Piero

@dezoito  Thanks
On MacOS, applications are typically added to /Applications, which seems writable, but the OS may be protecting to some degree. In any case though, not a great place to store data :)
By Apple's guidelines, most persistent, user-specific data that the user isn't expected to interact with outside of your app should be stored in an application specific directory in ~/Library/Application. The application specific subdirectory can just be the same as the name of your application bundle, though I think there is a more rigorous convention that is less likely to have collisions.
Data that the user might expect to use independently of your application should go into an app-specific or user chosen subdirectory in ~/Documents. For example, if your app generates text or json files with the results of a grid search that the user might want to use elsewhere.
Temp files should go in /tmp. There is an API for getting a unique, runtime-specific subdirectory.
Apple's documentation on the matter:
https://developer.apple.com/library/archive/documentation/FileManagement/Conceptual/FileSystemProgrammingGuide/FileSystemOverview/FileSystemOverview.html#//apple_ref/doc/uid/TP40010672-CH2-SW14
Often cross platform apps don't honor any of this and instead create an application specific directory in ~/. This is often prefixed by a period (ie .ollama-grid-search) so that it is hidden under ordinary circumstances.

@easp , @PieBru,
I tested each release installer for each platform and they all worked, but the only Mac I have available is still using MacOS 12.7.
Could you guys give this another and check if it now works for you as well?

On MacOS, applications are typically added to /Applications, which seems writable, but the OS may be protecting to some degree. In any case though, not a great place to store data :)

Yes, I was trying to save to a folder relative to the binary, regrettably.
Tauri has an API dedicated to storing application files, but understanding how it works wasn't very straight forward.
In case anyone else developing a Tauri app sees this, this is the issue that eventually helped me:
https://github.com/tauri-apps/tauri/discussions/5557
Again, I appreciate the effort in testing the app and helping me sort this issue.

No more errors now.
With only one model selected and more than one values in one or more params lists, I would expect more than one test will be performed, one for each "grid" of params. Instead:

You just discovered an input validation bug, thank you!
Try separating values by commas, please.

I did it with commas this morning, it works with commas.
Thank you!

Fixed the issue that was allowing space delimited lists to pass parameter validation.
(added a few discreet UI tweaks as well).
Closing this issue, but please LMK if any problems persist


FileTransform does not work on standalone json files?

It seems that the latest version of FileTransform requires a webpackage file and will not work on stand alone json files.  Would it be possible to update the behavior so that a webpackage is not required?

I have tried setting the folder path to

folderPath: '$(Build.SourcesDirectory)\tests**'
folderPath: '$(Build.SourcesDirectory)\tests***.json'

both of which result in an error
Here is the yaml I have tried
- task: FileTransform@2
  displayName: 'Transform appsettings.json'
  inputs:
    folderPath: '$(Build.SourcesDirectory)\tests\**'
    jsonTargetFiles: |
      $(Build.SourcesDirectory)\tests\**\appsettings.json
      $(Build.SourcesDirectory)\tests\**\appsettings.$(buildConfiguration).json

or
- task: FileTransform@2
  displayName: 'Transform appsettings.json'
  inputs:
    folderPath: '$(Build.SourcesDirectory)\tests\**'
    jsonTargetFiles: |
      appsettings.json
      appsettings.$(buildConfiguration).json

the result I get is
##[section]Starting: Transform appsettings.json
==============================================================================
Task         : File transform
Description  : Replace tokens with variable values in XML or JSON configuration files
Version      : 2.0.4
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/file-transform
==============================================================================
##[error]Error: More than one package matched with specified pattern: d:\a\1\s\tests\**. Please restrain the search pattern.
##[section]Finishing: Transform appsettings.json

Hi,
I see you've mentioned that "the latest version of FileTransform requires a web package file and will not work on stand-alone JSON files". However, file transform task V1 also always needed a web package file. This is not a new feature.

Thanks -- I realized that after I made the post.  If the intention is for this to only work with web packages that maybe the task should be renamed to "Web Package File Transformation"?

@ravensorb  We will try to make the task info more useful. Thanks for the feedback.

I'm a little confused as I am doing exactly this in a legacy pipeline, and it works fine. Here's the exported YAML:
steps:

task: FileTransform@1
displayName: 'File Transform: '
inputs:
folderPath: src/SkyJack.Agent
fileType: json
targetFiles: appsettings.json

Am I missing something? Is this only an issue with actual YAML pipelines?

@ravensorb I'm curious about the issue you ran into with this - I am using a File Transform task in multiple pipelines to transform standalone json files, including the appsettings.json file in a .NET Core project, and a standalone json file in a React app in another. The differences seem to be that I don't use wildcards in the folderPath and only specify a single file, which seems like it could be the issue from the error message you reported.

I believe you are correct -- it is the wildcard that is the issue.  I have since moved on to a different solution as I needed the ability to easily deal with the subfolders.  If the task is updated, happy to give it another try.

I believe you are correct -- it is the wildcard that is the issue. I have since moved on to a different solution as I needed the ability to easily deal with the subfolders. If the task is updated, happy to give it another try.

What did you find as a good alternative?


can't compile spontaneous_explode.sp

please fix or send pre-comiled version of it cuz i can't fix this
HELP PLS!!!!!!1

The releases section now has a release package with compiled plugins based off the most recent commit.


Camel-Telegram: Support for type "Document"

The telegram type "Document" is a general file (as opposed to photos, videos or audio files) and can be in any format. A Document can also be one of the already supported types, but then it's uncompressed (e.g. photos).
The benefits of using the Document type are:

Receiving and sending uncompressed files
Receiving and sending files of any format

This pull request contains the support for incoming and outgoing documents and unit tests.
Also see: https://core.telegram.org/bots/api#document

@nicolaferraro wonder if you could take a look as well.
@Neffez do you mind creating a JIRA ticket also so we can track this there too.

Here it is: https://issues.apache.org/jira/browse/CAMEL-10730

@Neffez thanks for the PR, I'll review it soon.

Great work @Neffez! (sorry for the delay)
The changes have been merged into master. I've just changed few bits and done a source check (mvn clean install -P sourcecheck on the component).
Do you mind closing this PR?

Cheers!


[Bug] [MGS2]:

Black screen when loading into the game (audio plays as normal). I am using the Lenovo Legion GO at 2560x1600 and have tried leaving the resolutions at the default 0 as well as inputting the resolutions in full. Issue does not seem to persist with MGS3 and I have had no problem with it on my gaming PC at 4K or the Steam Deck. I am using the current version of MGSHDFix.

MGSHDFix & Game Version Numbers: [v2.3]

Log Files:

Reproduction:

@Sastran0 log files?


Skip config if not in interactive mode

Since fzf.fish's config is sourced in non-interactive fish sessions, skipping unnecessary config will make non-interactive fish sessions a bit faster. This grealy benefits other things as well; for example tide makes use of non-interactive sessions extensively.

Great idea! I will merge this as soon as I figure out how to make it still possible to run tests locally. B/c right tests only work on CI but I run them locally when I'm developing.

Why not just pretend you are a CI (set -x CI true?)

Sure that'll work for me.

Actually, the tests still fail for me locally even with CI=true and set -x CI true

@kidonng can you help me answer these questions

I did s/subshell/session on your PR's description b/c it seems fish doesn't have subshells. Is this correct?
where in tide's code are non-interactive sessions run?
does this change affect command substitution?
where else are non-interactive sessions run?

I did s/subshell/session on your PR's description b/c it seems fish doesn't have subshells. Is this correct?

Subshell here is literally just shell as a subprocess. You can call it subprocess if you don't like it, but session is something entirely different.

where in tide's code are non-interactive sessions run?

fish_prompt. tide's code is straightforward so I guess I don't need to repeat it here.

does this change affect command substitution?
where else are non-interactive sessions run?

I'm not sure what you mean or what are referring to, and honestly I think they are out of scope of this PR.

I'm not sure what you mean or what are you referring to, and honestly I think they are out of scope for this PR.

I'm trying to learn these aspects of fish so I can properly write a summary of how this affects users' shell performance.

What is fish session then?

Ah funny that https://github.com/PatrickF1/fzf.fish/issues/181 was created the same day. Now I can add another reason this change is good: it makes fzf previews load a bit faster!

I'm not sure what you mean or what are you referring to, and honestly I think they are out of scope for this PR.

I'm trying to learn these aspects of fish so I can properly write a summary of how this affects users' shell performance.

🤔 Perhaps I should just give a short reply to those questions:

does this change affect command substitution?

That depends on what do you mean by "command substitution". But from my view, you don't usually start another fish in command substitution so no.

where else are non-interactive sessions run?

Again, are you referring to fzf.fish, tide or fish in general? I don't think I can give a better answer than reading the code/docs directly.

What is fish session then?

From starting a fish shell to exit this particular shell. In fact that's not a fish  terminology, not even a shell terminology, it's just session's  literal meaning.
There is something that I want to note, though. I consider "session" to imply an interactive launch. You don't think your shell scripts "start many sessions" right?

Ah funny that https://github.com/PatrickF1/fzf.fish/issues/181 was created the same day. I realized another reason this change is good: it makes fzf previews load a bit faster!

Hmm I don't think that issue has anything to with this change. Glad you are happy about it though.

Thanks for answering my questions.

Hmm I don't think that issue has anything to do with this change. Glad you are happy about it though.

It does doesn't it? When fzf loads a previews, it executes a new fish shell, which means it has to load the conf.d/fzf.fish in the process.

It does doesn't it? When fzf loads a previews, it executes a new fish shell, which means it has to load the conf.d/fzf.fish in the process.

That issue #181 has nothing to do with preview/shell launch speed.
The faster preview has been mentioned from the very beginning in this PR's description, until it is edited:

The faster preview has been mentioned from the very beginning in this PR's description, until it is edited

😅 Oops my bad. I think I was not mentally all there when I edited it. Sorry !


add register_bmi function to enable interoperablity with iso_c_bmif_2_0

This PR will enable the noah-mp bmi module to be useable by the ngen framework via the ISO_C_Binding BMI module (noaa-owp/ngen#282) and the fortran BMI adapter.
This does introduce a dependency on the iso_c_bmif_2_0 module, which can be found and built in the ngen subproject in ngen/extern/iso_c_fortran_bmi.  A PR on the ngen project with this library and the fortran adapter should be up shortly, will update this when it is.

I think there we can scheme up something that works.  The first two things that come to mind is implementing an independent module ngen_module which includes the noahmp_bmi and implements this function.  The other possibility is to use a preprocessor directive, and wrap the register_bmi function with #ifdef NGEN_ACTIVE or something like that.  The reason it doesn't compile is not finding the iso C module file.  So either of the approaches would give the ability for certain builds to not worry about it.

Have you talked to the CSDMS folks about the C-Fortran BMI extension? We could develop that as a standalone library that becomes a noah-mp-modular dependency if running in the framework.
I hesitate to merge the PR because it means the bmi implementation of surface will no longer work. Do you want to chat tomorrow about this?

So the only dependency of on the iso_c_bmif_2.0 in the register_bmi function is due to a type in the module, box that is a trivial wrapper type.  So another option is to add the type definition of box to the module alongside register_bmi and then there would be no external dependency required.  But I'm not sure what that would do to the iso_c_bmif_2.0 library that depends on that type.  It might work to duplicate it in both modules, but I'm not 100% sure about that at the moment.  Just wanted to save the idea here.

@SnowHydrology There is no rush to merge this, I just wanted to get it up for this exact discussion.  I think we can definitely solve this problem one way or another to make it possible to compile/run the module completely independently.  Once we get that, then we can merge.

@hellkite500 Sounds like a plan. Let me know when you want to move forward.

I agree with Keith, I'd say it's a requirement that the noah-mp-modular
versions compile and run both in standalone and bmi-connected modes, and we
probably shouldn't merge until we have that. I like the compiler directive
solution (enabling a single driver), but alternatively having bmi & non-bmi
drivers would also work.  The driver doesn't do much, so it should not be
hard to keep two relatively synced.  But either way, we should be able to
solve this.
Cheers,
-Andy
On Thu, Sep 2, 2021 at 1:30 PM K. Jennings @.***> wrote:

@hellkite500 https://github.com/hellkite500 Sounds like a plan. Let me
know when you want to move forward.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NOAA-OWP/noah-mp-modular/pull/12#issuecomment-911992096,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABIKARO4QYQAHQPOM2CBEPDT77GHBANCNFSM5C3TJS2Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

It may be better process to make a new branch for this in the OWP repo, then redirect this fork PR to that branch before finally merging it with master.  it may make it easier to reverse course (hopefully not needed but ...) and document the development changes, merge etc.

@SnowHydrology @andywood I implemented the CPP preprocessor option, so this should allow the code to compile stand alone (as long as NGEN_ACTIVE isn't defined to the preprocessor.  Seems to work well for now.

@hellkite500, It looks like we still need to add the new BMI specification to the /bmi directory and that information to the makefile.

@hellkite500, It looks like we still need to add the new BMI specification to the /bmi directory and that information to the makefile.

I don't understand, this should work entirely independent of the "new BMI specification".  I thought that was the intention.

@hellkite500, It looks like we still need to add the new BMI specification to the /bmi directory and that information to the makefile.

I don't understand, this should work entirely independent of the "new BMI specification". I thought that was the intention.

Probably just my confusion, but if someone sets NGEN_ACTIVE to true, then won't the module need access to bmif_2_0_iso? Or is NGEN_ACTIVE something we will only define when using the framework? If it's the latter, I'll just add a comment and approve the PR. Thanks!

if someone sets NGEN_ACTIVE to true, then won't the module need access to bmif_2_0_iso?

Yes, but the expectation is that the NGEN build system will set this flag and make the bmif_2_0_iso module for building the NGEN shared library.  For any "core" dependency, I see NGEN being responsible for compiling/linking external modules, such as noah-mp.  Here is what we do for the current noah-mp module, this CMake file builds the dependent iso module, sets the NGEN_ACTIVE cpp directive, and then builds the fortran sources for noah-mp and the surfacebmi module.  In this case noah-mp doesn't need to worry about the framework integration other than the few lines of code wrapped up in the cpp directives.
It is possible to replicate this behavior directly in the dependent modules.  If you want to be able to build a NGEN compatible shared library from your project's own build files, then you have to manage that iso dependency, which requires a more sophisticated build processes (auto conf, cmake, ninja, ect.) I don't think we want to get in the habit of copying source code files around too many places, but instead should try to manage them as real dependencies.

That's a good explanation and makes sense to me. I added two comment lines above the directive. Are you good with a merge?

I'm good, thanks for the comment commit!


symbols for each keyword in keywords timelines needs fixing

Currently there are some keywords that have multiple symbols rendered, this needs to be fixed so only one symbol is displayed for each keyword.

this relates to issue #57


Format of index.yaml for acr helm repo

I've notice that the format for url key in the generated index.yaml for acr helm repo uses relative paths. For instance, following the example at https://docs.microsoft.com/en-us/azure/container-registry/container-registry-helm-repos (to insert the sample wordpress chart) generates the following index.yaml cache. Notice that the url key is  _blobs/wordpress-5.2.5.tgz. This would have to be concatenated with the contextPath and the root domain to get the full URL.
apiVersion: v1
entries:
  wordpress:
  - acrMetadata:
      manifestDigest: sha256:08ef434162070bba4256414c80b001d15b7503ef2a1a4fa1f60bab174f80d4d7
    appVersion: 5.1.0
    created: "2019-03-06T16:59:25.8892193Z"
    description: Web publishing platform for building blogs and websites.
    digest: e0f315e609e02fd4798bed33d56267fd68f5e45dae2015b8895474178e6cc8e4
    engine: gotpl
    home: http://www.wordpress.com/
    icon: https://bitnami.com/assets/stacks/wordpress/img/wordpress-stack-220x234.png
    keywords:
    - wordpress
    - cms
    - blog
    - http
    - web
    - application
    - php
    maintainers:
    - email<EMAIL_ADDRESS>      name: Bitnami
    name: wordpress
    sources:
    - https://github.com/bitnami/bitnami-docker-wordpress
    urls:
    - _blobs/wordpress-5.2.5.tgz
    version: 5.2.5
generated: "2019-03-06T18:44:57Z"
serverInfo:
  contextPath: /helm/v1/repo

Is it possible to configure this to use full paths? I didn't find any options on the az CLI. This would be more inlined with how other repos like https://azure-samples.github.io/helm-charts/index.yaml do it.

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

ID: 2bed94c3-b7a7-40d4-0102-8c9b1f1145ca
Version Independent ID: 2d4ac8bd-dc7a-236c-56db-7f0e1ce0fd05
Content: az acr helm
Content Source: src/command_modules/azure-cli-acr/azure/cli/command_modules/acr/_help.py
Service: container-registry
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Hi, I am also facing the same issue. I want to edit this relative url as Rancher can't read this relative url to get to show the app catalog.

@toddysm Do you have any updates on this issue ?

Hi @vazexqi @S93rastogi  , not sure are you still facing the issue? Would you like to migrate to Helm 3?