JavaScript error: TypeError: Cannot read properties of undefined (reading 'call')

PHP Version
7.4.13
Shopware Version
<IP_ADDRESS>
Expected behaviour
Have vendor JavaScript work (all.js).
Actual behaviour
JavaScript has error which breaks all other JavaScript:

TypeError: Cannot read properties of undefined (reading 'call')

How to reproduce
Upgrade to the currently latest shopware version (<IP_ADDRESS>), clear cache, compile theme, open shop with console tab open (I was using Chrome).

Same here: https://github.com/shopware/platform/issues/152

I got this error because of a plugin. After deactivating it, the error disappeared. In my case it was the Google Translator Plugin by Bul Hinsche GmbH.

I got this error because of a plugin. After deactivating it, the error disappeared. In my case it was the Google Translator Plugin by Bul Hinsche GmbH.

Gonna try deactivating each plugin one after another.

I doubt this issue is on the plugin side.
After having tested this on two sites, the following plugins are "affected" as well:

Matomo
Packstation
Pixel Your Shop
Flexible popup management

I had to disable them all to get my JavaScript working again.
Is it really probable that all these plugins have a problem with 6.4.11.?
They all work fine with 6.4.10....

I doubt this issue is on the plugin side. After having tested this on two sites, the following plugins are "affected" as well:

Matomo
Packstation
Pixel Your Shop
Flexible popup management

I had to disable them all to get my JavaScript working again.
Is it really probable that all these plugins have a problem with 6.4.11.x? They all work fine with 6.4.10.x ...

I can confirm about matomo. BUT not all the time. I was testing a lot to figure out which one is the bad guy. And sometimes 4/5 enabled works fine, but after changing and returning back to the working 4, not.
Maybe somehow about the installation order or something?

Hi everyone,
I reopened this issue for the moment since it seems to be a general thing. We are investigating what exactly caused the problem and will get back to you asap. One of our partners suspected a change inside pseudo-modal.util.js which added new exports. I was able to reproduce this with a test plugin, however the error message in the console was a different one.
I attached the change with which I could fix the test plugin when applied to version <IP_ADDRESS>. But we will investigate further using your mentioned plugins. Sorry for the circumstances.
Index: src/Storefront/Resources/app/storefront/src/plugin/ajax-modal/ajax-modal.plugin.js
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/src/Storefront/Resources/app/storefront/src/plugin/ajax-modal/ajax-modal.plugin.js b/src/Storefront/Resources/app/storefront/src/plugin/ajax-modal/ajax-modal.plugin.js
--- a/src/Storefront/Resources/app/storefront/src/plugin/ajax-modal/ajax-modal.plugin.js	(revision 10086920a14d6c33e357054e51283993603888f8)
+++ b/src/Storefront/Resources/app/storefront/src/plugin/ajax-modal/ajax-modal.plugin.js	(date<PHONE_NUMBER>223)
@@ -4,7 +4,9 @@
 import LoadingIndicatorUtil from 'src/utility/loading-indicator/loading-indicator.util';
 import DeviceDetection from 'src/helper/device-detection.helper';
 import DomAccess from 'src/helper/dom-access.helper';
-import PseudoModalUtil, { PSEUDO_MODAL_TEMPLATE_CONTENT_CLASS } from 'src/utility/modal-extension/pseudo-modal.util';
+import PseudoModalUtil from 'src/utility/modal-extension/pseudo-modal.util';
+
+const PSEUDO_MODAL_TEMPLATE_CONTENT_CLASS = 'js-pseudo-modal-template-content-element';

 /**
  * This class extends the Bootstrap modal functionality by
Index: src/Storefront/Resources/app/storefront/src/utility/modal-extension/pseudo-modal.util.js
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/src/Storefront/Resources/app/storefront/src/utility/modal-extension/pseudo-modal.util.js b/src/Storefront/Resources/app/storefront/src/utility/modal-extension/pseudo-modal.util.js
--- a/src/Storefront/Resources/app/storefront/src/utility/modal-extension/pseudo-modal.util.js	(revision 10086920a14d6c33e357054e51283993603888f8)
+++ b/src/Storefront/Resources/app/storefront/src/utility/modal-extension/pseudo-modal.util.js	(date<PHONE_NUMBER>373)
@@ -2,10 +2,10 @@
 import Feature from 'src/helper/feature.helper';
 import { REMOVE_BACKDROP_DELAY } from 'src/utility/backdrop/backdrop.util';

-export const PSEUDO_MODAL_CLASS = 'js-pseudo-modal';
-export const PSEUDO_MODAL_TEMPLATE_CLASS = 'js-pseudo-modal-template';
-export const PSEUDO_MODAL_TEMPLATE_CONTENT_CLASS = 'js-pseudo-modal-template-content-element';
-export const PSEUDO_MODAL_TEMPLATE_TITLE_CLASS = 'js-pseudo-modal-template-title-element';
+const PSEUDO_MODAL_CLASS = 'js-pseudo-modal';
+const PSEUDO_MODAL_TEMPLATE_CLASS = 'js-pseudo-modal-template';
+const PSEUDO_MODAL_TEMPLATE_CONTENT_CLASS = 'js-pseudo-modal-template-content-element';
+const PSEUDO_MODAL_TEMPLATE_TITLE_CLASS = 'js-pseudo-modal-template-title-element';

 export default class PseudoModalUtil {
     constructor(

Hi
i had the same problem with SW <IP_ADDRESS> and this Plugin:
https://store.shopware.com/diga302677692244/conversion-tracking-google-ads-facebook-pinterest-microsoft....html
somehow, after i contacted the developer,  he managed to fix it in short time

@SorinSu do you know how the developer fixed it?
I had the same error on my local shopware instance.
I rebuild the script and it work fine now.
Maybe this help for you

If anyone stumbles upon this in the future:
I've made a solution-post in the shopware-forum because i didn't find anything of value for this yet
https://forum.shopware.com/t/all-js-uncaught-typeerror-cannot-read-properties-of-undefined-reading-call-nach-update-auf-6-4-11-1/94346/23


Need white theme

Please save my eyes. Thanks.

@celesteking at the moment I don't really have the time to think about this, sorry.


Choco install command suddenly became case sensitive!!!???

We used ver ...23 for a while, and updated to ...27 some weeks ago.
choco install packagename --source "my custom source"
choco install PackageName --source "my custom source"
These are not the same.. it should not care what case the package name is.

@codearoo can you let me know if this is still an issue with the latest prerelease of choco (put it on a canary system, it's choco.exe).

I'm going to call it fixed for now pending your response.

Yes this still is not fixed. I just tested on ver .32.

Sorry, I meant with version 0.9.9 (prerelease). Test on a non production
system please. It would be great if this can be closed with 0.9.9, I
believe it is fixed, I just don't have a source like proget that has this
issue.
On Wednesday, January 28, 2015, codearoo<EMAIL_ADDRESS>wrote:

Yes this still is not fixed. I just tested on ver .32.
—
Reply to this email directly or view it on GitHub
https://github.com/chocolatey/chocolatey/issues/589#issuecomment-71832010
.

--
Rob
"Be passionate in all you do"
http://devlicio.us/blogs/rob_reynolds
http://ferventcoder.com
http://twitter.com/ferventcoder

oh.. I don't know where to get that version.
would I do this?
choco install chocolatey -Version <IP_ADDRESS>  ??
I don't know where to see the list of valid beta versions.
On Wed, Jan 28, 2015 at 8:45 AM, Rob Reynolds<EMAIL_ADDRESS>wrote:

Sorry, I meant with version 0.9.9 (prerelease). Test on a non production
system please. It would be great if this can be closed with 0.9.9, I
believe it is fixed, I just don't have a source like proget that has this
issue.
On Wednesday, January 28, 2015, codearoo<EMAIL_ADDRESS>wrote:

Yes this still is not fixed. I just tested on ver .32.
—
Reply to this email directly or view it on GitHub
<
https://github.com/chocolatey/chocolatey/issues/589#issuecomment-71832010>

.

--
Rob
"Be passionate in all you do"
http://devlicio.us/blogs/rob_reynolds
http://ferventcoder.com
http://twitter.com/ferventcoder
—
Reply to this email directly or view it on GitHub
https://github.com/chocolatey/chocolatey/issues/589#issuecomment-71836431
.

You can find it here:
https://chocolatey.org/packages/chocolatey/0.9.9-alpha-20150125
choco install chocolatey -Pre

I don't know where to see the list of valid beta versions.

The versions of a particular package are shown at the bottom of each package page.

I didn't know how to find this alpha pre-release page.
anyway, I tried it with ver 0.9.9 and this issue seems resolved.
The install and removal of the pre-release version, however, is not working. :)  Installing with   "-pre" did work (with some errors) but then per instructions on the screen to --force <IP_ADDRESS>  failed. So now that test system can't get out of pre-release.

I didn't know how to find this alpha pre-release page.

From any package version page, i.e:
https://chocolatey.org/packages/chocolatey/<IP_ADDRESS>
Scroll to the bottom, and you will see:

Where you can then see all available package versions, including pre-release.

Why I asked for you not to do it on a prod system. Downgrading requires
removing some files, although you can probably just use the
chocolatey.org/install.ps1 script and it should work.
On Wednesday, January 28, 2015, Gary Ewan Park<EMAIL_ADDRESS>wrote:

I didn't know how to find this alpha pre-release page.
From any package version page, i.e:
https://chocolatey.org/packages/chocolatey/<IP_ADDRESS>
Scroll to the bottom, and you will see:
[image: image]
https://cloud.githubusercontent.com/assets/1271146/5938971/b46c68e2-a6f7-11e4-8f01-32e884b8651c.png
Where you can then see all available package versions, including
pre-release.
—
Reply to this email directly or view it on GitHub
https://github.com/chocolatey/chocolatey/issues/589#issuecomment-71840294
.

--
Rob
"Be passionate in all you do"
http://devlicio.us/blogs/rob_reynolds
http://ferventcoder.com
http://twitter.com/ferventcoder

It was a test system..but still want to get it back.. and thought I should
let you know
I'll brute force it if your suggestion does not work.

Let me know if it doesn't work. It should. I haven't tried a choco install chocolatey --version <IP_ADDRESS> --force

Although doing exactly that seems to have worked for me.

Update: replaced image... for some reason the last capture cut off the bottom

Interestingly enough, I would have thought there would be more output from the install.

choco install chocolatey --force --version <IP_ADDRESS>
that worked...  somehow when I ran it I omited the   "--version" part... it
was doing something... not sure what then.
On Wed, Jan 28, 2015 at 10:07 AM, Rob Reynolds<EMAIL_ADDRESS>wrote:

Although doing exactly that seems to have worked for me.
[image: image]
https://cloud.githubusercontent.com/assets/63502/5939894/127e0574-a6cd-11e4-99f1-bd8c26932b1e.png
—
Reply to this email directly or view it on GitHub
https://github.com/chocolatey/chocolatey/issues/589#issuecomment-71849336
.


Phpstan issues

Changes

Upgrade phpstan version
Fix issues reported by phpstan

Issue
https://github.com/deployphp/deployer/issues/2996

[x] Bug fix #…?

[ ] New feature?

[ ] BC breaks?

[ ] Tests added?

[ ] Docs added?
Please, regenerate docs by running next command:
$ php bin/docgen

Both unit and e2e tests are failing.

Both unit and e2e tests are failing.

True :thinking: But I see they were already failing in the latest master: https://github.com/deployphp/deployer/runs/5413879503?check_suite_focus=true. It seems to be separate issues, don't you think?

Ohh, true. Forgot to merge https://github.com/deployphp/deployer/pull/2995

Can you rebase on master, please?


Publishing timeline?

Have been updating a few packages and have hit a roadblock here since color has been removed from the core. Is there anything more that needs to be done with this guy before it gets put up on packages?

Can you share what you need it for? That can help me prioritize because I'm not sure if it will be in the same form it was before.

Specifically pukkamustard/elm-identicon which uses the rgb and hsl converters. It's quite a small subset of requirements; so for now I could just generate a color submodule for elm-identicon and let you run free on the changes you have envisioned for this package. Does that sound like the best plan for the interim?

I've used it in combination with the elm-color-extra package for projects in 0.18. It has functions for converting between colorspaces as well as blending, interpolating, and building gradients for colors.
That library is used in two of my projects — one for a closed-source example where we make heavy use of gradients, and one in an open-source project where it's used rather minimally for its colorToHex function.

I could be wrong, but it looks like style-elements relies on this package, so not being able to use this with 0.19 limits its usage with  style-elements.

Can someone make a package like what they want? The old API is pretty simple. The new API is pretty simple. I do not think the math in this repo is complete or correct in all cases, and this is definitely not something where I have unique expertise on the topic. So I think it'd be better to not block just because this might exist someday.

Sure, no problem. I can take that on. May be a few days before I can start work on it though. That is, unless someone else in this thread is keen?
If not, I'll ping you all later with what I've got so far and we can move further discussions to a new tracker.

Thanks @Libbum, let me know if you want some help with it.

For people that stumble across this: It might be worth pointing out that the efforts have moved to https://github.com/avh4/elm-color/issues/ where you can come comment and help with building a great standard-worthy colour module.


Perfomance increas

changed the for loop to copy data to a memcpy for performance benefits

Hi,
thx for pointing this out. You're absolutely right. BUT this function isn't used anyway so I just removed it :+1:
Kind regards,
Max


Add suggestion to NewJobForm to let customer know how to minimize gas fees

It would be great to let customers know how to minimize cost of gas fees while creating a new task on the platform.
We can mention, that customers may attach links to Google Docs or GitHub (using our Markdown markup, for example) with detailed explanation of their tasks and minimize transaction fees.
Please provide your ideas about this task before implementing this feature.
Reference: https://github.com/optriment/optrispace-frontend-v2/blob/master/src/forms/NewJobForm/index.jsx#L358

#43


Recommend Paratext check and inventories to teams.

Paratext includes many basic checks for consistency and USFM structure.  The report can be overwhelming though when run with all checks on all books.  However some issues in our drafting process may be avoided, or performance of the models may improve if those checks are run and issues fixed.
Often a whole class of issues can be fixed in one go by editing correctly the various inventories such as:
Valid Characters, Valid Punctuation, Markers Inventory. The Markers Inventory in particular may help find rare markers that are not required by the project, or markers in the wrong place.
It would be good to find ways to encourage teams to make use of these, especially where there is a clear benefit in terms of model performance.

Could this be part of the onboarding process?  We may be able to detect that certain checks have or have not been run and then encourage the team to run them.

Yes we could mention this during the onboarding process. If we can run them or not I don't know - we probably don't want to rewrite them, though that wouldn't be terrible.   We should focus on those issues that have an impact on the quality of the models and inferences.


Hide the command bar

 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.


Plugin is crashing the harvester

On import phase, the following gets thrown by geoview:
$ paster --plugin=ckanext-harvest harvester import --config /etc/ckan/default/development.ini
/usr/lib/ckan/default/src/ckan/ckan/new_authz.py:6: FutureWarning: ckan.new_authz has been renamed to ckan.authz. The ckan.new_authz module will be removed in a future release.
  FutureWarning)
2015-10-17 09:06:50,410 DEBUG [ckanext.spatial.model.package_extent] Spatial tables defined in memory
2015-10-17 09:06:50,417 DEBUG [ckanext.spatial.model.package_extent] Spatial tables already exist
2015-10-17 09:06:50,431 DEBUG [ckanext.harvest.model] Harvest tables defined in memory
2015-10-17 09:06:50,432 DEBUG [ckanext.harvest.model] Harvest tables already exist

2015-10-17 09:06:50,478 DEBUG [ckanext.harvest.model] Harvest tables already exist
DB tables created
2015-10-17 09:06:50,483 INFO  [ckanext.harvest.logic.action.update] Harvest objects import: {'source_id': None, 'package_id': False, 'harvest_object_id': False}
2015-10-17 09:06:50,494 DEBUG [ckanext.dcat.harvesters.rdf] In DCATRDFHarvester import_stage
/usr/lib/ckan/default/local/lib64/python2.7/site-packages/sqlalchemy/orm/unitofwork.py:79: SAWarning: Usage of the 'related attribute set' operation is not currently supported within the execution stage of the flush process. Results may not be consistent.  Consider using alternative event listeners or connection-level operations instead.
  sess._flush_warning("related attribute set")
2015-10-17 09:06:50,599 DEBUG [ckanext.spatial.plugin] Received: u'{"type": "Polygon", "coordinates": [[[-180.0, -90.0], [180.0, -90.0], [180.0, 90.0], [-180.0, 90.0], [-180.0, -90.0]]]}'
2015-10-17 09:06:50,610 DEBUG [ckanext.spatial.lib] Extent for package 1e85a8ef-efcc-4fa1-a40a-3cc1bec5c8bc unchanged
Traceback (most recent call last):
  File "/usr/lib/ckan/default/bin/paster", line 9, in <module>
    load_entry_point('PasteScript==1.7.5', 'console_scripts', 'paster')()
  File "/usr/lib/ckan/default/local/lib/python2.7/site-packages/paste/script/command.py", line 104, in run
    invoke(command, command_name, options, args[1:])
  File "/usr/lib/ckan/default/local/lib/python2.7/site-packages/paste/script/command.py", line 143, in invoke
    exit_code = runner.run(args)
  File "/usr/lib/ckan/default/local/lib/python2.7/site-packages/paste/script/command.py", line 238, in run
    result = self.command()
  File "/usr/lib/ckan/default/src/ckanext-harvest/ckanext/harvest/commands/harvester.py", line 152, in command
    self.import_stage()
  File "/usr/lib/ckan/default/src/ckanext-harvest/ckanext/harvest/commands/harvester.py", line 325, in import_stage
    'package_id': self.options.package_id,
  File "/usr/lib/ckan/default/src/ckan/ckan/logic/__init__.py", line 429, in wrapped
    result = _action(context, data_dict, **kw)
  File "/usr/lib/ckan/default/src/ckanext-harvest/ckanext/harvest/logic/action/update.py", line 273, in harvest_objects_import
    harvester.import_stage(obj)
  File "/usr/lib/ckan/default/src/ckanext-dcat/ckanext/dcat/harvesters/rdf.py", line 284, in import_stage
    p.toolkit.get_action('package_update')(context, dataset)
  File "/usr/lib/ckan/default/src/ckan/ckan/logic/__init__.py", line 429, in wrapped
    result = _action(context, data_dict, **kw)
  File "/usr/lib/ckan/default/src/ckan/ckan/logic/action/update.py", line 371, in package_update
    {'package': data})
  File "/usr/lib/ckan/default/src/ckan/ckan/logic/__init__.py", line 429, in wrapped
    result = _action(context, data_dict, **kw)
  File "/usr/lib/ckan/default/src/ckan/ckan/logic/action/create.py", line 468, in package_create_default_resource_views
    create_datastore_views=create_datastore_views)
  File "/usr/lib/ckan/default/src/ckan/ckan/lib/datapreview.py", line 303, in add_views_to_dataset_resources
    create_datastore_views)
  File "/usr/lib/ckan/default/src/ckan/ckan/lib/datapreview.py", line 261, in add_views_to_resource
    'package': dataset_dict
  File "/usr/lib/ckan/default/src/ckanext-geoview/ckanext/geoview/plugin.py", line 279, in can_view
    format_lower = resource['format'].lower()
KeyError: 'format'

And I have no clue how to recover from that, since none of the clear or purge operations seem to work since the DB entries are now in an inconsistent state somehow. When I try to clear jobs I get:
An error occurred: [(IntegrityError) update or delete on table "package" violates foreign key constraint "harvest_object_package_id_fkey" on table "harvest_object" DETAIL: Key (id)=(1e85a8ef-efcc-4fa1-a40a-3cc1bec5c8bc) is still referenced from table "harvest_object". "begin; \n update package set state = 'to_delete' where id in (select package_id from harvest_object where harvest_source_id = 'fd15d7cd-b576-4577-acbb-7246eee9b41e');\n delete from resource_view where resource_id in (select id from resource where package_id in (select id from package where state = 'to_delete' ));\n delete from resource_revision where package_id in (select id from package where state = 'to_delete' );\n delete from resource where package_id in (select id from package where state = 'to_delete' );\n \n delete from harvest_object_error where harvest_object_id in (select id from harvest_object where harvest_source_id = 'fd15d7cd-b576-4577-acbb-7246eee9b41e');\n delete from harvest_object_extra where harvest_object_id in (select id from harvest_object where harvest_source_id = 'fd15d7cd-b576-4577-acbb-7246eee9b41e');\n delete from harvest_object where harvest_source_id = 'fd15d7cd-b576-4577-acbb-7246eee9b41e';\n delete from harvest_gather_error where harvest_job_id in (select id from harvest_job where source_id = 'fd15d7cd-b576-4577-acbb-7246eee9b41e');\n delete from harvest_job where source_id = 'fd15d7cd-b576-4577-acbb-7246eee9b41e';\n delete from package_role where package_id in (select id from package where state = 'to_delete' );\n delete from user_object_role where id not in (select user_object_role_id from package_role) and context = 'Package';\n delete from package_tag_revision where package_id in (select id from package where state = 'to_delete');\n delete from member_revision where table_id in (select id from package where state = 'to_delete');\n delete from package_extra_revision where package_id in (select id from package where state = 'to_delete');\n delete from package_revision where id in (select id from package where state = 'to_delete');\n delete from package_tag where package_id in (select id from package where state = 'to_delete');\n delete from package_extra where package_id in (select id from package where state = 'to_delete');\n delete from package_relationship_revision where subject_package_id in (select id from package where state = 'to_delete');\n delete from package_relationship_revision where object_package_id in (select id from package where state = 'to_delete');\n delete from package_relationship where subject_package_id in (select id from package where state = 'to_delete');\n delete from package_relationship where object_package_id in (select id from package where state = 'to_delete');\n delete from member where table_id in (select id from package where state = 'to_delete');\n delete from related_dataset where dataset_id in (select id from package where state = 'to_delete');\n delete from related where id in ('');\n delete from package where id in (select id from package where state = 'to_delete');\n commit;\n " {}]

The original crash was fixed on 5cdd81c. This might not have been published on a new version on PyPi, I'll do that later today. In the meantime you can run the latest source.
As for the harvester issues let's discuss them on its repo.

Please look at my pull request. Your fix is only one half of it. I wrote the other half ;)


stdlib: re-add include paths for swift headers

This adds the swift include path manually to the builds for the stubs
and the runtime.  This has no impact for the build currently.  However,
adding the additional include directory will enable a standalone build
for the stdlib.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test and merge


Spi features

Hello, i wrote some new functions to spi. Inspired from multiple spi drivers i had like this one : https://github.com/MetalPhreak/ESP8266_SPI_Driver.
New funtions:
spi_set_command(uint8_t bus,uint8_t bits, uint16_t data);
spi_set_address(uint8_t bus,uint8_t bits, uint32_t data);
spi_set_dummy_bits(uint8_t bus, uint8_t bits, bool pos);
spi_clear_address(uint8_t bus);
spi_clear_command(uint8_t bus);
spi_clear_dummy(uint8_t bus);
void spi_repeat_send_8(uint8_t bus, uint8_t data, int32_t repeats);
void spi_repeat_send_16(uint8_t bus, uint16_t data, int32_t repeats);
void spi_repeat_send_32(uint8_t bus, uint32_t data, int32_t repeats);

I tested these functions on the oscilloscope. all work fine but you can do some test to check.
I had some strange behavior with memcpy so i used this code,
SPI(bus).W[i] = *dword;
instead of,
memcpy((void *)&SPI(bus).W[i], dword, SPI_32BIT);
Maybe need test again ?
It will allow SPI3 protocol for SSD1306 driver and more.

Hi @Zaltora,
Thanks for the contribution.

I had some strange behavior with memcpy so i used this code,
SPI(bus).W[i] = *dword;
instead of,
memcpy((void *)&SPI(bus).W[i], dword, SPI_32BIT);

The problem with memcpy might be that it discards volatile qualifier of SPI(bus).W which causes that memw instructions are not placed before accessing registers.
I noticed tabs in the code, we don't use tabs in eor.
Overall looks like a useful addition, but I haven't checked the code in details yet.

Sry about tab, forget to change my auto-formater in eclipse. BSD style come back !

Very useful functions! Repeating send is needed in most SPI protocols.
Going to merge it tomorrow if no objections.
@Zaltora Thank you, great contribution!


undefined error in classicui-limit-parsys.js

Required Information

[ ] AEM Version, including Service Packs, Cumulative Fix Packs -6.4.2
[ ] ACS AEM Commons Version: _____ 4.1.0
[ ] Reproducible on Latest? yes/no - yes

This issue was raised many times earlier but not fixed yet. EX: https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/1759
Could you please fix this asap as editors are not able to drop any components in parsys?
Expected Behavior
Should be able to drag & drop the components from sidekick on to the parsys.
Actual Behavior
Drag & drop the component from the sidekick to the pasrys isn't working in classic UI and there is console error from this file https://github.com/Adobe-Consulting-Services/acs-aem-commons/blob/master/content/src/main/content/jcr_root/apps/acs-commons/widgets/configure-limit-parsys/classicui-limit-parsys.js#L96.
It looks like sometimes the "designObject" is not initialized within the pageInfo and just returns as an undefined limit object.

Steps to Reproduce

Start AEM 6.4.2
Install ACS commons 3.17.4 or the latest.
Install the attached content package.
Open the page http://localhost:4502/content/parsys-drag-drop-issue.html
Drag & drop the component to the parsys to see the console error. Some times it works, so try refreshing the window or try in the incognito window.

@anand19cse are you explicitly including the ACS Commons classicUI Widgets clientlibs? Or in other words, are you trying to use this feature? Or is this appearing simply because ACS AEM Commons is installed?

Hi David,
Thanks for the response. But latest version of ACS commons has the fixes. I installed the latest version and it was fixed.
Thanks,
Anand


Stop mentioning ConsoleProgressMonitor.jl

as TerminalLoggers.jl supersedes it.
ref https://github.com/tkf/ConsoleProgressMonitor.jl/issues/9

Codecov Report

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

@@           Coverage Diff           @@
##           master      #22   +/-   ##
=======================================
  Coverage   71.77%   71.77%
=======================================
  Files           1        1
  Lines         163      163
=======================================
  Hits          117      117
  Misses         46       46

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 be50902...1796b07. Read the comment docs.


thread related crash

I'm currently setting up a scoreboard system.
often at random times i get a thread error thrown at me
Called from sys.thread._Thread.HaxeThread::create C:\HaxeToolkit\haxe\std/cpp/_std/sys/thread/Thread.hx line 130 Critical Error: Uncatchable Throw: Null Object Reference
my current setup for this is nothing more than
GameJolt.fetchScore(10, tableID, null, null, null, null, null,  {
            onSucceed: function(response:Dynamic):Void
                {
                    for (i in 0...response.scores.length)
                    {
                        var userName:FlxText = new FlxText(150, 100 + 70 * i, 400, '#${i+1} ${response.scores[i].user}', 16);
                        add(userName);
                    }
                },
                onFail: function(message:String):Void
                {
                    trace(message);
                }
        });

Try not creating the text within the thread, instead, change only the text within it.

ah, yeah that seemed to work, thanks!


Desktop app availability

Thank you for building Aonsoku – its UI is one of the best I’ve seen among Navidrome clients! 🎉
I was wondering if you have any estimated timeline for releasing a downloadable desktop app? I’d be happy to lend a hand if there’s any way I can assist with it. 💐
Also, I have a small suggestion: it might be more intuitive to move the language dropdown into the user menu. Switching languages isn’t something most users do frequently, so it might fit better there.
Thanks again for your great work!

Thank you so much for the kind words, I’m thrilled to hear that you enjoyed the UI!
Regarding your first question, I haven’t released desktop versions of the app yet, mainly to avoid exceeding the GitHub Actions free tier. However, I’m considering making these available in future releases, so stay tuned!
As for your suggestion about moving the language dropdown to the user menu, that’s a great idea, and I’ll be implementing it in upcoming versions. I'm thinking about moving the theme switch too.
Thanks again for reaching out!


getting blank image  instead of actual results

@pmneila
hi!
i am trying this on my spyder but i am not getting the results instead i am getting the blank image.
Also tell me the steps to execute it smoothly.

Hello @syedsajidhussain,
I'm going to need more information about your problem. Have you tried running it in the terminal? Does it work there? If it works in the terminal, the problem might be related to spyder and not to morphsnakes. How does this "blank image" looks? What code are you running exactly? And what do you mean by "execute it smoothly"?

i tried to run examples.py through windows terminal itz showing the following output:
C:\Users\user\Desktop\morphsnakes-master\morphsnakes-master>python examples.py
WARNING:root:No display found. Using non-interactive Agg backend.
examples.py:61: UserWarning: Matplotlib is currently using agg, which is a non-GUI backend, so cannot show the figure.
plt.pause(0.001)
examples.py:282: UserWarning: Matplotlib is currently using agg, which is a non-GUI backend, so cannot show the figure.
plt.show()

@pmneila

@pmneila
now after running again through conda terminal i am getting the output as follows:

Hi @syedsajidhussain,
Try editing examples.py and commenting out the lines
if os.environ.get('DISPLAY', '') == '':
    logging.warning('No display found. Using non-interactive Agg backend.')
    matplotlib.use('Agg')

Then, run examples.py from the terminal. It should work now. If it works, you can try it in Spyder. I don't have much experience with Spyder, but it looks like it uses a non-interactive matplotlib backend by default. If you really need to run the code from Spyder and want to see the evolution of the contours of morphsnakes, you need to set up Spyder to use an interactive backend. You might find useful information to do so in this Stack Overflow question.
Let me know if it works.


Fix the vercheck program

Vercheck is currently broken, and needs to get fully implemented.

Tested, and it works!


Support for aeson-<IP_ADDRESS>

aeson-<IP_ADDRESS> is now available with lts-8.1 resolver.
Now any (probably any) failed webdriver command throws BadJSON instead of FailedCommand.
λ> runSession defaultConfig $ openPage "https://google.com" >> (findElem $ ByXPath $ "//body")
Element "0"
λ> runSession defaultConfig $ openPage "https://google.com" >> (findElem $ ByXPath $ pack "//something_that_does_not_exist")
*** Exception: BadJSON "Word is either floating or will cause over or underflow: -2.0"

<3


helm upgrade will ignore crds

https://github.com/everoute/everoute/pull/702
upgrade cni should set crd's annotation to manager by helm mantually to upgrade


Fix chaotic chinese characters with golang cli commands

When debugging load command, it is found that ./bin/alluxio CLI can't process Chinese characters correctly. There must be something wrong with the CLI encoding or decoding. It seems that the Chinese characters are encoded incorrectly.
This happens because the construction of CLI commands does not initialize environmental variables correctly. OS-level environmental variables are ignored, so the program uses a false encoder.
This pull request fixes this problem.

i think the java subprocess should inherit the environment variables from the go CLI process. it does so by default but not when we change the Env field. https://pkg.go.dev/os/exec#Cmd. @Xenorith

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-3-1.1.x please


Fixed the Build for you

I didn't watch the stream but saw your post on Twitter. Here's a working deployment. I setup Azure Static Web app in my account to text with my own api token and it worked.
Multiple jobs were running async and independently. I just merged all jobs and removed "dotnet-version" arg from the deploy and set it explicitly in the deploy step. It was being inherited from the global env var and that’s why it was picking up 3.1.404.
The next problem was that the action expects a package.json “build” or “build:azure" command to run but you can override it I think according to the docs since you really don’t want to build it again. (See here: https://docs.microsoft.com/en-us/azure/static-web-apps/github-actions-workflow#build-and-deploy)
So essentially you're defining build steps for your client app inside package.json. It can be done manually but as of right now, the Github actions expects this workflow.

Not sure why the build is failing with api token missing. It works on my repo when I put the correct secret in.

closing since it was manually pulled in


AttributeError: 'NoneType' object has no attribute '_parameters'

Hi, thanks for this work.
When I used higher for a cutomized network inheritated from torch.nn.Module, it reported the following error when it excutated  at higher.innerloop_ctx(self.online_net, inner_opt, copy_initial_weights=False) as (fnet, diffopt):
AttributeError: 'NoneType' object has no attribute '_parameters'

I can print the network with .parameters. What could be the problem? Thanks.

My problem. Close it.

I love it when problems solve themselves 😄


Review

Thanks for the module! 🙂 I took a look at the code and suggest a few changes.
There are no functional changes.

Thanks! All the changes look good to me so I'll go ahead and merge the changes 👍


[authoring] pressing escape in markdown editor removes toolbar

There's no way to get it back other than closing and opening the plank again or reloading.
If you do it for one editor, it impacts all editor planks.

I can't repro this anymore. Please open again if anyone encounters this.


(feat) Restorer: investigate allowing child removal to trigger focus restoration in Safari & Firefox

Related to #305 -- with the fix in #306, Restorer will work in Safari & Firefox if the root node itself is removed from the DOM, but not if a child node containing document.activeElement is removed (note - CSS changes like display: none will still work, only DOM removal is an issue).
This seems like more of a minor use case/feature to support, since the primary use case is likely removing the root node. This issue is for tracking & to not lose the edge case after #306 is closed. Discussed w/ @ling1726

Using focusout to detect child removal will still be supported for chromium based browsers - no change there


Download PDF of ASP.NET Core docs returns pdf with duplicate pages (and consequently wrong table of contents page pointers)

https://docs.microsoft.com/en-us/aspnet/core/introduction-to-aspnet-core?view=aspnetcore-3.1
Clicking the "Download PDF" of ASP.NET Core docs returns a pdf with duplicate pages (and consequently wrong table of contents page pointers).
For example, try browsing for Handle JSON Patch Requests under Web API section. The table of contents page is wrong, and if you manually browse the contents for the topic you will find duplicate pages.

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

ID: c165f8e4-d058-27b2-ee9b-1bbed0996f1f
Version Independent ID: faed8bb0-79ef-7af4-5deb-e7645cdf2e39
Content: Introduction to ASP.NET Core
Content Source: aspnetcore/introduction-to-aspnet-core.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

This should be reported at MicrosoftDocs-feedback.


Continuous function InnerSimplified

This is for https://github.com/asc-community/AngouriMath/issues/428 , but I have only done this for continuous functions because it's sort of unclear.

Codecov Report

Merging #455 (9e5b2bf) into master (78c6f99) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #455      +/-   ##
==========================================
- Coverage   81.04%   81.01%   -0.04%
==========================================
  Files         150      150
  Lines       12964    12964
  Branches     1775     1775
==========================================
- Hits        10507    10503       -4
- Misses       1924     1926       +2
- Partials      533      535       +2

Impacted Files
Coverage Δ

...inuous/Solvers/EquationSolver/ExponentialSolver.cs
92.64% <0.00%> (-2.95%)
:arrow_down:

...Solvers/EquationSolver/AnalyticalEquationSolver.cs
81.25% <0.00%> (-2.09%)
:arrow_down:

Continue to review full report at Codecov.

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

What's unclear?

Should InnerSimplified be added only to continuous functions like in the example in the issue, or should they also be extended to logic and all areas?

Yeah, should be applied to logic and other functions too. If the user wants to run a function which doesn't simplify anything implicitly, they will just use functions from MathS.

Thanks for your contribution!


Publish mcu-debug.memview to Open VSX

Dear extension author,
Please publish this extension to the Open VSX marketplace.
Context
Unfortunately, as Microsoft prohibits usages of the Microsoft marketplace by any other products or redistribution of .vsix files from it, in order to use VS Code extensions in non-Microsoft products, we kindly ask that you take ownership of the VS Code extension namespace in Open VSX and publish this extension on Open VSX.
What is Open VSX? Why does it exist?
Open VSX is a vendor neutral alternative to the MS marketplace used by most other derivatives of VS Code like VSCodium, Gitpod, OpenVSCode, Theia-based IDEs, and so on.
You can read on about Open VSX at the Eclipse Foundation's Open VSX FAQ.
How can you publish to Open VSX?
The docs to publish an extension can be found here. This process is straightforward and shouldn't take too long. Essentially, you need an authentication token and to execute the ovsx publish command to publish your extension. There's also a doc explaining the whole process with an example GitHub Action workflow.

👆 This message template was used for the issue.
I have already taken care of everything. Only your approval is missing. Thank you!

Published it a few days ago. Will be doing it every time I do a release for VSCode
https://open-vsx.org/extension/mcu-debug/memory-view


Docker files updates

This PR updates the docker-compose file so that:

MongoDB can be used as a backend when running BigchainDB server.
The websocket events API can be used.

Note for @ttmc this will most likely require minor changes to docs. Will add you as a reviewer once the docs have been updated.

[x] update docs

Codecov Report

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

@@          Coverage Diff          @@
##           master   #286   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           9      9
  Lines         220    220
=====================================
  Hits          220    220

I like the simpler docker-compose file, with db and bdb-server. I wonder if bdb isn't a little too general, though, since it's actually the driver that it's running.

I wonder if bdb isn't a little too general, though, since it's actually the driver that it's running.

This is from a client point of view. We have been using bdb in the client's docs to refer to an instance of the client for example.


Added more themes to the editor and language support

We are currently limited by the Code Editor package we use by relying on a third party to update it so instead we add it to the local packages and edit it from there, giving us control over the editor. I also add some extra theme choices for the editor:
Xcode

Github

Google Code

Visual Studio

Also added support for more languages (Check Language in Modules/CodeEditor).

I thing @MarcoCarnevali is already working on something similar?

I thing @MarcoCarnevali is already working on something similar?
#36 (comment)

Could possibly be yes.


ServiceMonitor doens't work out of the box

Hello. I've deployed the dragonfly-operator Helm chart with the following values:
serviceMonitor:
  enabled: true

But my VMAgent can't scrape metrics:
unexpected status code returned when scraping "http://<IP_ADDRESS>:8443/metrics": 400; expecting 200; response body: "Client sent an HTTP request to an HTTPS server.\n"

Thanks for reporting!
I can try testing it out, but in the meanwhile we have something tested around metric collection of Dragonfly at https://www.dragonflydb.io/docs/managing-dragonfly/operator/prometheus-guide

We are facing the same problem.
As far as I see, the ServiceMonitor is targeting the port 8443 where as in the pod configuration the parameter --metrics-bind-address=<IP_ADDRESS>:8080 is set with a other port.

Also see: https://github.com/dragonflydb/dragonfly-operator/issues/242

Need:

start operator pod with -metrics-bind-address=<IP_ADDRESS>:8080
this port(8080) must set in ports of deployment
serviceMonitor is targeted to this port

And why don't you use podMonitor instead serviceMonitor? This operator deployment doesn't seem to require the service, as it doesn't accept traffic.


Unable to install using Brew

Hi, I keep getting this error

Those commands look right and I'm not able to reproduce your issue. Maybe try troubleshooting brew? https://docs.brew.sh/Troubleshooting

I reinstalled it everything including Homebrew, but now I am getting the same error that caused me to reinstall everything

Not sure what happened with your brew earlier but it looks fine now. The issue you're currently seeing is #49.


Revert "Update repositories.txt"

Reverts arduino/library-registry#670

@ArduinoToolingBot

@ArduinoBot

@ArduinoBot

Hello @ProjectoOfficial. As mentioned by the bot, this type of pull request can't be handled by the automated system as is done for submissions. But I'm happy to help out.
I need a little more information in order to do that. Please tell me what is the result you want to achieve via this pull request.

Hello @ProjectoOfficial. I'm checking back in with you to see if I can move this forward. If you no longer wish to make this change, I can close the PR in order to keep the issue tracker tidy.

Closing due to lack of response.
@ProjectoOfficial if you will provide the requested information by commenting here, I'll be happy to re-open this PR and move forward with the work on it.


KAFKA-4772: Exploit #peek to implement #print() and other methods

I remove KeyValuePrinter and KStreamForeach two class, then implements them by KStreamPeek.
So, now KStreamPeek can do KeyValuePrinter and KStreamForeach job.

Left a meta-comment, otherwise lgtm.

@guozhangwang I'm done :)

I think Yes :)

LGTM and merged to trunk. Thanks for your contribution @jedichien !

Created https://issues.apache.org/jira/browse/KAFKA-5267

@guozhangwang @mjsax thank you for reviewing, this is my first merge. I cannot wait to do more :)

@guozhangwang @mjsax thank you for reviewing, this is my first merge. I cannot wait to do more :)

Congrats!


Add a task to run the nomulus tool

Fixes b/140447455.

This change is

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @CydeWeys and @jianglai)

core/build.gradle, line 596 at r1 (raw file):

// To run the nomulus tool:
// gradle registryTool --args="foo --bar"

There's no better way to handle args? Wrapping (and escaping) args into another arg is ugly.

See cl/267645678 as an internal solution. Using --args is the recommended way to pass args (https://docs.gradle.org/current/userguide/application_plugin.html#sec:application_usage).

Reviewable status: 0 of 1 files reviewed, 2 unresolved discussions (waiting on @CydeWeys and @jianglai)

core/build.gradle, line 600 at r1 (raw file):

  classpath = sourceSets.main.runtimeClasspath
  main = 'google.registry.tools.RegistryTool'
}

Unless you anticipate using the tool from within the build, it might be better to do this with a (very small) shell script. Gradle has a lot of startup overhead and adds a lot of extraneous output, and this approach likely wouldn't work at all for "nomulus shell" or anything that consumes standard input.

The purpose of this task is to add the ability to easily test changes to the nomulus tool, not day-to-day operations.

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @jianglai)

core/build.gradle, line 596 at r1 (raw file):
Previously, jianglai (Lai Jiang) wrote…
Ugh. I don't like it, but if there's no alternative ... :/
So this doesn't solve the problem of replacing what we used to have, which is a simple target to run a locally built version of nomulus tool. Can we write a new bashrc target that handles --args and escaping so that we can run a locally built tool easily?

Isn’t cl/267645678 exactly what you described...?


Add tuple spread parsing and typechecking

This doesn't implement all of union/intersection handling, that object spread have

/cc @jbrown215


OpenVPN Instance - does not register Network Interfaces in Unbound settings

Description
The new OpenVPN Instance does not register a Network Interface in the general Unbound settings.
Currently you need to configure to listen on all interfaces to get DNS working on the new OpenVPN Instances
Steps to reproduce the behavior:

Create OpenVPN instance
Go to General Settings of Unbound
OpenVPN instance is not visible in the Network Interface list (OpenVPN server (old) are visible.

Expected behavior
OpenVPN Instance is to be expected to be selectable on the Network Interface List in the Unbound setting.
Environment
OPNsense 23.7.2

Closing the ticket as incomplete and open new one.


When code runner extension gets installed, disable the play button icon immediately

Use extensions.onDidChange to figure out if code runner extension is installed, and hide the play button icon immediately.
#7423

With the "Code Runner" extension installed, using the latest insiders, I only see the icon provided by the other extension.  Ours no longer shows.  If I uninstall the "Code Runner" extension then our icon shows up again.


Typo in the dependency import

I realize that the current published library compile path is
compile 'com.jaeger.statusbaruitl:library:1.3.6'
But the statusbaruitl is intended to be statusbarutil if I am not misinterpreting. Any plan to correct it or leave it be?

I am sorry for this mistake, but change it is difficult. I will find a right way to modify this mistake.

Thank you for getting back to me. No worry at all.

I have fix typo, version 1.4.0 is published
compile 'com.jaeger.statusbarutil:library:1.4.0'


Remove old docs completely [SA-14289]

Built on #77
The old docs are not required at all anymore: all traces of them can be removed.

Superseded by #79


return correct gRPC error codes

closes #497

This PR includes some whitespace changes to common/common_test.go because my IDE (vscode) automatically formats files, and an earlier commit to this file used a different formatting convention. I recommend viewing the diff with whitespace ignored (click on the gear symbol near the top of the Files changed (diff) page).


--disable-encryption not working anymore

Hi,
There is currently an issue with the patch to disable encryption to allow portability (Eloston/ungoogled-chromium#591) on Windows. Looks like this regression occurs between Chromium 76.0.3809.132 to Chromium 77.0.3865.75 (see https://github.com/brave/brave-core/pull/3385#issuecomment-545787343 and portapps/brave-portable#33). Do you have any idea about this @tangalbert919 @Eloston? I would like to fix this on Brave and ungoogled-chromium but hard to find something relevant. I have this output when I open ungoogled-chromium (80.0.3987.149) if --disable-encryption is enabled:
[1896:2800:0323/163555.011:ERROR:os_crypt_win.cc(102)] Failed to decrypt: Clé non valide pour l’utilisation dans l’état spécifié. (0x8009000B)

Thanks

This issue should be closed if it is already fixed.

Is this fixed? This issue is over a year old. I'm also struggling to recall the details of this feature, so I'm not sure how to properly test if it is still working.

No, this issue still exists. Please refer to the details https://github.com/Eloston/ungoogled-chromium/issues/1239. #1239

This has yet to be fixed in

ungoogled-chromium 92.0.4515.159

This has yet to be fixed in ungoogled-chromium 92.0.4515.159 @Eloston

This problem basically leads to non-working portable version: once I disable encryption - no cookies are saved because machine id is disabled

@kleuter I cannot reproduce your problem. To me, the original issue described by OP has been resolved. I have set ---disable-encryption and --disable-machine-id and my browser is perfectly portable. Successfully tested by me and a few other folks I came across at another forum.
Please note that by default, ungoogled-chromium clears cookies and website data when exiting. I would suggest that you disable such option in chrome://settings and try again.

Not true, try running your 'portable' app on another machine.
https://portapps.io/app/ungoogled-chromium-portable/#known-issues

Doesn't have to even be on an different machine ... Unfortunately when using these flags even if deleting all cookies on exit is disabled the cookies, etc. are still lost after closing and restarting the browser.
Still happens on the most recent version of ungoogled-chromium on Windows 10.

@kleuter As I mentioned, I and a few other folks already tested across different machines. I configured my browser on a Win11 machine and then moved it to another Win10 laptop. Extensions and cookies are intact. As a user of portable applications for 10+ years, I know perfectly what "portable" means.
The build I'm using is from winchrome
@leo-liar I don't know what you meant by "most recent version". If you are referring to the same portapps build that @kleuter mentioned, then it is not the most recent as it only offers chromium 97. winchrome offers up to chromium 98.
General principle: Main stream git repo does not maintain -- hence is not responsible for -- 3rd-party builds. portapps is a 3rd-party build. Therefore, if you wanted to report any issue in this git repo, you could (and should) have tried some other builds to if it is a universal issue, or even better, built your own executable.

I was referring to these to builds:

ungoogled-chromium_97.0.4692.71-1.1_windows by nifury
Chromium v98.0.4758.80-r950365 - ungoogled by macchrome/Marmaduke

Both show the aforementioned bug of deleting all cookies despite configured to not to do so if both disable encryption/machine-id flags are set.
It's true that apps, extensions and settings are portable but cookies are not -- as several other users already pointed out in some of the referenced issues.

@leo-liar Thanks for the info. Then I won't be able to help here, because I and my online acquaintances are using the exact same build your are using (the 2nd link you posted) and we cannot reproduce this issue. ungoogled-chromium is my daily driver, if this issue of deleting cookies exists for me, I would be mad.
The only possibility I can think of is that there is some scripts/applications which accidentally delete the cookies file, because in 98 it changes location (to be inside the Network folder, rather than the root profile folder). Other than that, sorry I cannot help further.

Ah, it's good to know that you do not have this issue while using the most recent version. I was under the impression that this bug occured in all recent (Windows) versions of ungoogled-chromium.
I don't think any script/application is messing with my profile folder. As I specify the actual profile folder via command line switch I'm not to worried about (upcoming) changes to it's default location either.
I'll try to create a fresh profile and test if this bug still occurs. #58 got me thinking that this might be related to having a rather old profile which has been kept over several versions of ungoogled-chromium (some of which may hae been buggy in regards to the aforementioned flags).

Just to clarify - I'm not saying the location of the [profile] folder (could be [user-data-dir]\Default, or [user-data-dir]\Profile 1, [user-data-dir]\Profile 2... based on your use case) is changed. I'm saying the Cookies file has moved. For example, prior to 97, it likely resides as [user-data-dir]\Default\Cookies, but after 98 it is [user-data-dir]\Default\**Network**\Cookies
And yes please try a new profile. That's a good first-step for trouble-shooting.

After creating a new profile and testing around (and observing that you were right - cookies and log-in states weren't deleted upon browser exit with a fresh profile) in the end it was just enough for me to delete the Local State file from my original profile folder and cookies, etc. are suddenly surviving a browser exit/restart now!
Aftwerwards I just had to reenable some flags that were configured via chrome://flags/ menu (instead of via command line) but no other settings were lost.
(Apparently there is now a different key here "os_crypt":{"encrypted_key":"%BASE_64_KEY%"} in Local State but no other relevant changes. And I haven't tested actually moving this profile to another computer yet.)

@leo-liar I'm glad that you tested and that it turned out to work for you too. Cheers.
@Eloston I suggest to close this issue for now.

Just for completeness:
I tested using my profile on another computer which worked as well. That means that full portability (at least while using the --disable-encryption and --disable-machine-id flags and disabling cookie deletion in the settings) can be achieved.
Though it may be necessary to delete the Local State file if the used profile is already older (i.e. from a previous ungoogled-chromium version).

Ok, --disable-encryption and --disable-machine-id flags work IF used from the command line. When used from chrome:flags - all cookies are deleted

Caused by the "Disable encryption" flag. All works fine with only the "Disable machine ID" flag

So I tried passing  --disable-encryption and --disable-machine-id flags in all sorts of ways and configurations moving between systems to test checking the database files. Nothing works.
Is there any progress on the issue I would love a truly portable chrome.

Did you pass the flags via command line or via shortcut?
And did you remove any previous existing Local State file?
And within the browser settings, are cookies, etc. to be kept on browser exit?

Command line.
Yes.
Yes.
A simple test where you run:
chrome.exe --user-data-dir=profile --disable-encryption --disable-machine-id
Then check the "Login Data" database look in the logins table and check the password element column for saved passwords they will still be encrypted.

i'm trying to move to a new windows install, chrome is the last thing I need to backup, and then this issues of tieing everything to a machine was brought up.
so here is my issue, I have 0 idea what chrome has backed up on their servers, however I have extensions that I consider required to even use chrome, imagezoomer being one of the main ones, along with a number of scripts and settings. show of taking several hundred screenshots and rebuilding if chrome doesn't restore everything, I was hoping dragging and dropping files would work, but apparently that doesn't work with chrome because of the encryption.
is there any way to use ungoogled chromium to get these files ported or am I 100% at the whims of what googled decided to backup on their end?

@Hashnoob
Have you tried this with a separate, newly created portable install?
Regarding the enrypted table values: I'm not sure that using said flags leads to unencrypted login information being stored. I think that the used encryption (maybe just some basic encoding?) will be independent of OS user accounts (and hardware) thus making it portable.
I've just checked this myself - USER column entries (in the Login Data sqlite file) are cleartext but passwords are blobs. The profile is still portable though.

Will this be ever fixed? :/
Don't want to switch to Firefox x)

For me the only thing that was hard keeping me on chrome just works on firefox too, it was just never submitted to easily be installed on it, so for me i'm trying to get a portable version of firefox going because next time I want to move from one os to another I don't want to have to sit here for 2 months wondering how to deal with chromes bs.

I don't know if this has been mentioned before as I did not read the complete discussion, but have you tried deleting your chromium settings folder from the disk before using the flag? Maybe there is an encrypted file present from a previous run which then failed to decrypt as no decryption key is used.

With #disable-machine-id, #disable-encryption enabled, to make cookies not get lost every reset, this worked for me:

Backup your chrome://flags
Exit Chromium
Backup your profile folders (User Data\Default, ...)
Empty out User Data
Launch chrome.exe with --disable-machine-id --disable-encryption
Enable chrome://flags/#disable-machine-id, chrome://flags/#disable-encryption
Restore your backed up chrome://flags
Disable Clear cookies and site data when you close all windows in chrome://settings/cookies
Exit Chromium
Delete User Data\Default and restore your backed up profile folders
Launch Chromium as usual (no need for the flags anymore)

(am unsure if anything other than flags is stored in non-profile files in User Data and should be backed up as well)
Seems like Chromium's first launch has to be with --disable-machine-id --disable-encryption in order to create a state in User Data in which cookies do not get lost.
Hopefully this helps narrow down the issue.

okay, i just tested this a bit.
turns out that you should use --user-data-dir=profile --disable-encryption --disable-machine-id on shortcut ,in your first use before you even have a user profile if you plan to backup or use it for portable, remember to disable clear cookies on exit.
if you enable it on flags without using --disable-encryption --disable-machine-id on shortcut before creating profile, you might encounter cookies delete even if you disable clearing cookies on exit.
Also, dont switch off you switch off --disable-encryption --disable-machine-id, it will clear cookies and reset so don't remove it

With #disable-machine-id, #disable-encryption enabled, to make cookies not get lost every reset, this worked for me:
(you should backup your whole User Data folder before trying this)  Should I enable --disable-machine-id --disable-encryption prior to backup or mac's if needed and user data otherwise the backup is encrpted?

Backup your chrome://flags How do you do this, the script I found no longer works..Trust issue
Exit Chromium
Backup your profile folders (User Data\Default, ...)  Did you also export PreferenceMACs regkey or is that not necessary to repopulate the extensions?
Empty out User Data Just to confirm, leave the folder, delete everything inside correct?
Launch chrome.exe with --disable-machine-id --disable-encryption Should I define the profile folder as suggested below like this --user-data-dir=%LOCALAPPDATA%\Chromium\User Data\Default or --user-data-dir=C:\Users\JP\AppData\Local\Chromium\User Data\Default
Enable chrome://flags/#disable-machine-id, chrome://flags/#disable-encryption
Restore your backed up chrome://flags
Disable Clear cookies and site data when you close all windows in chrome://settings/cookies
Exit Chromium
Delete User Data\Default and restore your backed up profile folders
Launch Chromium as usual (no need for the flags anymore)  So after following all the steps we still have to use flags in cli as suggesed below then remove if it worked?

(am unsure if anything other than flags is stored in non-profile files in User Data and should be backed up as well)
Seems like Chromium's first launch has to be with --disable-machine-id --disable-encryption in order to create a state in User Data in which cookies do not get lost.
Hopefully this helps narrow down the issue.

I have found the same through trial and error. Need to create a new profile with options, then it works.


PrereleaseIdentifiers with the different path

Description
Hello, I have a problem with versioning packages. I have 2 tags with the same major and minor versions. Also, I have two different prereleaseIdentifiers. I have a problem in that I always get only the higher path uploaded tag also when I have a different prereleaseIdentifier.
Example:

My tags look like this:

1.0.202303011159-someAlpha
1.0.202303011150-someBeta

Implementation:
.package(url: "someURL.git", .upToNextMinor(from: Version(1, 0, 0, prereleaseIdentifiers: ["someBeta"]))),
In this case, I get this package - 1.0.202303011159-someAlpha
Expected behavior
1.0.202303011150-someBeta
Actual behavior
I get 1.0.202303011159-someAlpha

I think this actually behaves correctly? Specifying the prerelease identifier opts you into getting any prerelease in the specified range, not a particular concrete prerelease identifier. cc @tomerd

So is there any way how we can get the newest version of the tag with specific prereleaseIdentifier?
Eg.
1.0.202303011159-someAlpha
1.0.202303011150-someBeta
My package will look like this:
.package(url: "someURL.git", .upToNextMinor(from: Version(1, 0, 0, prereleaseIdentifiers: ["someBeta"])))
And I will get:
1.0.202303011150-someBeta
I thought that prereleaseIdentifiers says specific tag with specific suffix in this case -suffix

Any progress here? Is this actually the expected behaviour of the prerelease identifiers? @DavidKadlcek @neonichu


Does not work on iPhones

Hey @chr15m,
I visited your project https://chr15m.github.io/barcamp-whats-the-point-of-lisp/ on the phone, but wasn't able to click through.
Device tested with: iPhone 13

So, I decided to tinker a bit with the code ... and if you like, I would love to contribute and open a PR ✌
My conclusion:
The listeners on the body did not fire, as the body seems to not be targeted properly when clicking/tapping in mobile Safari. I added a possibility to check the event target against:
... - we might have to work on the css a little to make it work in portrait mode... and in landscape mode I would try to stretch #main from left to right taking over the full width. That's just my first thought tbh.
The new helper function:
(defn main-dom-element [document]
  (first (.getElementsByTagName document "main")))

modifying tap:
(defn tap
  [ev]
  (when (or (= (aget ev "target") (aget js/document "body"))
            (= (aget ev "target") (main-dom-element js/document))) ;; 👈
    (let [x (aget ev "clientX")
          w (aget js/window "innerWidth")]
      (if (< x (/ w 2))
        (swap! state update :slide dec)
        (swap! state update :slide inc)))))

I'd be happy to help 👍

Either flashing yes or maybe with css-grid can help with this 🤔
I will investigate a little and get back to you! I envision it something like this, when on a touch device:

<

>

optional FOOTER

You could add a new component here: https://github.com/chr15m/scittle-tiny-slides/blob/main/slides.cljs#L69 called e.g. [component:touch-ui] and that can render the left right tappable areas on the screen positioned absolutely and with sufficient size. If there is a touch event outside those two tappable areas then they could appear briefly and fade or disappear after a second. If they are tapped correctly we don't need to show them I guess.


Action space to directly control vehicle acceleration

Is your feature request related to a problem? Please describe.
Driving datasets like NGSIM only contain driving trajectories without action information. But for most imitation methods, we need expert action guidance. Since we can not directly recover brake and throttle from trajectories, the current continuous agent type can not be used.
Describe the solution you'd like
A new action space that includes the acceleration and angular velocity is needed so that we can compute the expert action from the expert states(e.g., position, speed, and heading at each timestep).

@zbzhu99:  we are going to try to get this implemented in the next few days.

@zbzhue99, we just merged PR #854 into the develop branch for this.  Please check it out and let us know if that's what you had in mind.

@zbzhu99, we just merged PR #854 into the develop branch for this. Please check it out and let us know if that's what you had in mind.

Thank you. And when will PR #827 be merged?

@zbzhu99 I'm still waiting on a second review on it (our policy is that every merge should have 2), but if I haven't gotten one by the time I get the second review on #836, I will merge it anyway (since we discussed it in a meeting here, which sort of counts as a review).  Hopefully that will be tonight, as Iman said he could do it.

Note:  PR #878 changes the semantics of this action space such that acceleration is now expected to be a 3-vector instead of a scalar.


Fail Note 20 N981U

{
.name = "Samsung Galaxy Note 20",
.model = "SM-N981U",
.android_version = 12,
.android_security_patch.year = 2022,
.android_security_patch.month = 9,
.kernel_version = KERNEL_VERSION(4, 19, 113),
.ram_offset = 0x28000000UL,
}
QS1: Is the exploit only working on Kernel Version 5,x,x?
QS2: How can I get the ram+offset?
c1q:/ $ LD_PRELOAD=/data/local/tmp/libbadspin.so sleep 1
Bad Spin Exploit (CVE-2022-20421) by 0xkol
[x] Looking for binder_proc's inner_lock offset
[x] Trigger vulnerability... (mode = 1)
[x] Trigger use-after-free
[x] Finish spinning at spin_lock()
[x] Trigger vulnerability... (mode = 1)
[x] Trigger use-after-free
[x] Finish spinning at spin_lock()
[x] Trigger vulnerability... (mode = 1)
[x] Trigger use-after-free
[x] Finish spinning at spin_lock()
[x] Trigger vulnerability... (mode = 1)
[x] Trigger use-after-free
[x] Finish spinning at spin_lock()
[x] Trigger vulnerability... (mode = 1)
[x] Trigger use-after-free
[x] Finish spinning at spin_lock()
[x] Trigger vulnerability... (mode = 1)

The exploit works on devices running on kernel versions 5.4.x and 5.10.x only

Great Job. But is is possible to work on pixel 6 emulator?


Add Docker + cloud-init Terraform module to list

I really appreciate the attention that went into writing the contribution guidelines. Takes the guesswork out of what to expect from a PR 🙂
I'd like to contribute a link to a Terraform module I've recently created that bootstraps a VM with Docker + its associated config using cloud-init. The config it generates can be used with any of the major cloud platforms (thanks to cloud-init) and includes some useful extras like Traefik for automating the provision of SSL certificates through Let's Encrypt.
Unfortunately it doesn't really lend itself to a snappy title. I'd welcome any feedback you have!

Hi @christippett, thank you for contributing to this list.
It is true that the naming thing isn't the easiest thing to do. But it got me thinking, If you can gather more terraform modules related to Docket, we could create a section and put a few there.
What do you think ?

@christippett ping


Move get/setConfigJson to Config

Moved methods, getConfigJson and setConfigJson, from KernelManager to
Config.

Looks good to me :+1:


Error: File already exists in database: google/protobuf/any.proto

OS: Ubuntu 20.04
ROS: foxy

I placed your simulation package into my src directory and replaced the 'Hokuyo laser scanner' completely with the 'mid360' in my original file 'robot.xacro'.
<xacro:include filename="$(find ros2_livox_simulation)/urdf/mid360.xacro" />
<xacro:mid360 name="livox" parent="base_link" topic="mid360">
    <origin xyz="0.125 0.0 0.225" rpy="0 0 0"/>
</xacro:mid360>

After running colcon build, there were no errors, but when I attempted to run the simulation, I can generate models in Gazebo successfully, but there is no topic from mid360. And I encountered an error:
[spawn_entity.py-4] [INFO] [1696239884.345127553] [spawn_entity]: Spawn status: SpawnEntity: Successfully spawned entity [robot]
[gzserver-1] [libprotobuf ERROR google/protobuf/descriptor_database.cc:58] File already exists in database: google/protobuf/any.proto
[gzserver-1] [libprotobuf FATAL google/protobuf/descriptor.cc:1358] CHECK failed: GeneratedDatabase()->Add(encoded_file_descriptor, size):
[gzserver-1] terminate called after throwing an instance of 'google::protobuf::FatalException'
[gzserver-1]   what():  CHECK failed: GeneratedDatabase()->Add(encoded_file_descriptor, size):
[INFO] [spawn_entity.py-4]: process has finished cleanly [pid 45564]
[gzserver-1] Aborted (core dumped)
[ERROR] [gzserver-1]: process has died [pid 45556, exit code 134, cmd 'gzserver /home/lihanchen/Coding/PB_RMSimulation_ws/install/pb_rm_simulation/share/pb_rm_simulation/world/RMUC2023_world/RMUC2023_world.world                                                                      -s libgazebo_ros_init.so   -s libgazebo_ros_factory.so   -s libgazebo_ros_force_system.so       '].

I check my protoc version:
~$ protoc --version
libprotoc 3.6.1

I noticed that you have the following line in your CMakeLists:
target_link_libraries(ros2_livox libprotobuf.so.9)

However, when I checked in /usr/local/lib, I only found libprotobuf.so.10, libprotobuf.so.31, and libprotobuf.so.32. I tried changing the version number of libprotobuf.so in the CMakeLists to match my computer, but it didn't work.
I would like to know how to resolve this. I would greatly appreciate it if you could help me.

I have already solved the problem by uninstalling protobuf and then reinstalling it.


fix: add plugins to library import file

Prevent Xcode build error due to missing relationship between library import file and pubspec.yaml.
Added all plugins to library import file.
Fix: https://github.com/casdoor/casdoor-flutter-sdk/pull/33#issuecomment-1763629162

@fischermario fix CI first:

Replaced by #35


'ZIAClientHelper' object has no attribute 'config'

Describe the bug

'ZIAClientHelper' object has no attribute 'config' exception is returned when trying to activate saved configuration

Steps to reproduce

Screenshots

Your Environment

Collection: zscaler.ziacloud
Python:  3.10.12
Ansible: 2.17.3
Zscaler SDK Python Library & version (e.g. zscaler-sdk-python 0.1.1): zscaler-sdk-python    0.9.0

@mragaei Please indicate which version of the collection you're currently using.
Also, make sure you have followed the pre-requisites described in the following help page: by installing the SDK prior to executing your code.
https://zscaler.github.io/zpacloud-ansible/requirements.html
Based on our tests, everything is working normally.
Zscaler DevRel


Enhancements for the MqttPacketIdentifierProvider

Describe the feature request
The scenario is to store the messages with packet identifier for QoS>0 on the filesystem before publishing them to the MQTT Broker. Storing the messages on the filesystem helps to survive the system/mqtt client crashes.
Currently the MQTTNet does not expose any method to set the PacketIdentifier directly in the MqttApplicationMessage and the PacketIdentifier is only available in the response of PublishAsync method.
In the reconnection scenario, Due to the Reset() method from MqttPacketIdentifierProvider being called in the ConnectAsync() the packet identifier is reset to 1, if there are messages already stored in the filesystem from the last connection that will then cause the packet id conflict.
Which project is your feature request related to?

Client

Describe the solution you'd like
Following are possible solutions.

Introducing Get property in the MqttPacketIdentifierProvider so that the current value could be read.
Making the call to Reset() method optional in ConnectAsync() method.

Another approach for the point 1 would be to make it possible to directly assign the packet identifier in the MqttApplicationMessage and create the interface for MqttPacketIdentifierProvider so user can provide custom packet identifier provider.

I prefer to expose the packet identifier in the MqttApplicationMessage. If it is left 0 a new one will be created using the identifier provider. Additionally, I will add a new parameter to the client options which allows to setup the offset of the identifier values when connecting with the server. The client will also get a new property which exposes (read only) the latest used packet identifier so that you can store it after disconnecting and reuse it when connecting. What do you think?

Yes, that covers the use case. Thank you!


Fresh install of Ballista 11 crate fails with dependency errors

Describe the bug
Installing the crate gives dependency errors:
error: failed to select a version for `quote`.
    ... required by package `syn v2.0.0`
    ... which satisfies dependency `syn = "^2"` of package `zerocopy-derive v0.7.3`
    ... which satisfies dependency `zerocopy-derive = "=0.7.3"` of package `zerocopy v0.7.3`
    ... which satisfies dependency `zerocopy = "^0.7.0"` of package `ahash v0.8.4`
    ... which satisfies dependency `ahash = "^0.8"` of package `ballista-core v0.11.0`
    ... which satisfies dependency `ballista-core = "^0.11.0"` of package `ballista v0.11.0`
    ... which satisfies dependency `ballista = "^0.11.0"` of package `t v0.1.0 (/t)`
versions that meet the requirements `^1.0.25` are: 1.0.33, 1.0.32, 1.0.31, 1.0.30, 1.0.29, 1.0.28, 1.0.27, 1.0.26

all possible versions conflict with previously selected packages.

  previously selected package `quote v1.0.10`
    ... which satisfies dependency `quote = "^1.0.10"` of package `zerocopy-derive v0.7.3`
    ... which satisfies dependency `zerocopy-derive = "=0.7.3"` of package `zerocopy v0.7.3`
    ... which satisfies dependency `zerocopy = "^0.7.0"` of package `ahash v0.8.4`
    ... which satisfies dependency `ahash = "^0.8"` of package `ballista-core v0.11.0`
    ... which satisfies dependency `ballista-core = "^0.11.0"` of package `ballista v0.11.0`
    ... which satisfies dependency `ballista = "^0.11.0"` of package `t v0.1.0 (/t)`

failed to select a version for `quote` which could resolve this conflict

To Reproduce
docker run --rm rust bash -c "cargo new t && cd t && cargo add ballista"

FWIW, we are going to be releasing version 0.12 shortly.
https://github.com/apache/arrow-ballista/issues/926

I am closing this issue now that Ballista 0.12.0 has been released.


Loading glTF while bevy window is not active freezes bevy

Bevy version
main branch c4b8210
Operating system & version
Fedora 33 using Wayland
What you did
Tried to load an glTF, while waiting I clicked in another windown to wait;
What you expected to happen
The glTF to be loaded and spawned;
What actually happened
Nothing, I do think bevy froze, gnome asked to kill the program because it wasn't responding.
Additional information
This bug sometimes happens sometimes doesn't, I did launch bevy from the VScode terminal if that makes any difference;
Alt+Tab back doesn't unfreeze bevy;

Would you have any stack traces or anything that was printed for better context?

@lassade Is this isolated to only some models? If so, could you link or upload one where you see this occurring?

This bug sometimes happens sometimes doesn't,

That sounds like either a system ordering issue or a malformed asset that we're not handling properly.

I can't reproduce, and this hasn't been reported recently.
Please reopen if you have more information on how to reproduce


Fix xcodebuild to actually work on Darwin even without ZAP installed.

Darwin builds without ZAP installation fails and this failure is not detected by CI.
Changes:

narrow down the scoping of environment variables on darwin, so that this kind of bug does not happen again
Generate a "pre-generate" directory structure for darwin specifically, so that xcode builds can make use of chip_code_pre_generated_directory variable
update pregenerate logic to allow narrowing down the pre-generated code. Effectively allows a minimal subset of files to be generated for darwing (e.g. we do not need java and bridge)
update zap_generate_all to also generate the darwin pregenerated directory

Please update summary to explain what was wrong and the fix. I am trying to understand the code by review, however there seem to be several changes like env location updates, codegen logic updates and xcode updates. I would like to understand so if we ever change xcode again, this kind of error does not occur anymore.

I added a bit of a description here, however I am not sure about the cpp-app codegen. Why do we codegen those, it was not needed before. We are going down the wrong path if we start adding more codegen pregen.

Please update summary to explain what was wrong and the fix.

I thought the summary covered this already, under "Changes", but I tried to make it clearer a bit what the problem was.
To be clear, I view this as a temp fix to unblock us for now, until we fix the "libCHIP.a depends on codegen" bug in the build system.

however there seem to be several changes like env location updates

That's "narrow down the scoping of environment variables on darwin, so that this kind of bug does not happen again", first item under "Changes".

codegen logic updates

That's the third and fourth item under "Changes".

and xcode updates.

Second item under "Changes": using chip_code_pre_generated_directory and the paths to the headers changed.

I would like to understand so if we ever change xcode again, this kind of error does not occur anymore.

See first item under "Changes".


[Idea] Selective Disclosure with JWT-SD

Is your feature request related to a problem? Please describe.
We don't have a method to do selective disclosure!
Describe the solution you'd like
Using https://www.ietf.org/archive/id/draft-ietf-oauth-selective-disclosure-jwt-04.htmlto add in a simple selective disclosure scheme.
Describe alternatives you've considered
BBS+, SPARTAN.
BBS+ has some impls, but needs substantial work/research to integrate here.
SPARTAN is still a WIP.
Getting it done
Nice to have, not immediately urgent. Happy to take this myself.

Closing as fixed by #288 and #369


new docker hub image for v4.23.4

the v2fly/v2ray-core has a new v4.23.4 release, so we need to build the docker image accordingly

PR is here


Relocate filter toggles for reduced screen clutter

[x] Move "Show All Buildings" to main nav and ~rename as Property Type~
[x] Make the timeline a dropdown

See discussion on the request to "rename as property type" below.
┆Issue is synchronized with this Asana task

It's been fairly easy to get here, but I'm also remembering some discussion with the client about other options (mutli-select, or a dropdown containing checkboxes, etc.)

Right now these are just straight-up selects, using "Selectize" - an ancient synthetic dropdown generator, which is already in use in the application.
If instead, these should be more like the Neighborhood and Council dropdowns, which are kind of custom dropdown/combo-boxes, then more discussion is needed

Note from mike:

Year selector option should list year options descending, so most recent year comes first.

One issue I just noticed with the building type selector. When "High-rise Multifamily" is selected, it doesn't fit:

This is all set


gortr: 0.14.6 -> 0.14.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/cloudflare/gortr/releases.
meta.description for gortr is: "The RPKI-to-Router server used at Cloudflare"
meta.homepage for gortr is: "https://github.com/cloudflare/gortr/"
meta.changelog for gortr is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

directory tree listing: https://gist.github.com/e6bdef5fb761c5339c6afe4c341e336d

du listing: https://gist.github.com/704122816586ff3ad8d571ba3de2eb22

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
gortr

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ykvpq2i32r55ixhps63hxyn8md458v5n-gortr-0.14.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gortr https://github.com/r-ryantm/nixpkgs/archive/beb87dff736d681fd5fbc1ea1b46b6cecfef1908.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ykvpq2i32r55ixhps63hxyn8md458v5n-gortr-0.14.7
ls -la /nix/store/ykvpq2i32r55ixhps63hxyn8md458v5n-gortr-0.14.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:

gortr

Maintainer pings
cc @petabyteboy @kalbasit for testing.

Result of nixpkgs-review pr 104981 run on x86_64-darwin 1

1 package built:

gortr


General Issue: Restructure Files for ADEG and ADVS Parameters

Background Information
The files derive_adeg_parms.R and derive_advs_parms.R should be restructured such that there is one file per "parameter". E.g., derive_param_qtc.R contains derive_param_qtc() and compute_qtc().
The test files should be restructured accordingly.
Definition of Done
derive_adeg_parms.R and derive_advs_parms.R and their test files are replaced.

@pharmaverse/admiral @pharmaverse/admiral_comm great issue to dig into the files of admiral and get familiar with some some of the functions.  This is probably 1 hour of work and might produce some "uuummmm...moments" that we will need to discuss in the Pull Request.

Hi @sadchla-codes ,
are you taking care of this issue or should I take it? Sorry I missed last {admiral} core dev meeting, since I attended the PHUSE EU Connect 2024.

@bms63 @jeffreyad
It seems that the task involves transferring both the function code and its documentation from the original R files to the new R files, as shown in the table below:

Original File
Function
New File

derive_adeg_params.R
derive_param_qtc
derive_param_qtc.R

derive_adeg_params.R
default_qtc_paramcd
derive_param_qtc.R

derive_adeg_params.R
compute_qtc
derive_param_qtc.R

derive_adeg_params.R
derive_param_rr
derive_param_rr.R

derive_adeg_params.R
compute_rr
derive_param_rr.R

------------------------
----------------------
---------------------

derive_advs_params.R
derive_param_map
derive_param_map.R

derive_advs_params.R
compute_map
derive_param_map.R

derive_advs_params.R
derive_param_bsa
derive_param_bsa.R

derive_advs_params.R
compute_bsa
derive_param_bsa.R

derive_advs_params.R
derive_param_bmi
derive_param_bmi.R

derive_advs_params.R
compute_bmi
derive_param_bmi.R

@bms63 @jeffreyad It seems that the task involves transferring both the function code and its documentation from the original R files to the new R files, as shown in the table below:

Yes, and also change the test files test-derive_advs_params.R and test-derive_adeg_params.R the same way

Love this table.
@bundfussr and @manciniedoardo should we split the compute functions into separate files we sometimes do this and sometimes do not.
@luenhchang good to review this section as well as you are exploring admiral more

@bundfussr and @manciniedoardo should we split the compute functions into separate files we sometimes do this and sometimes do not.

I wouldn't put the compute functions into separate files because the derive and compute function are so closely connected.

@bms63 @jeffreyad @bundfussr
I have created the following new files:

R/derive_param_bmi.R
R/derive_param_bsa.R
R/derive_param_map.R
R/derive_param_qtc.R
R/derive_param_rr.R
tests/testthat/test-derive_param_bmi.R
tests/testthat/test-derive_param_bsa.R
tests/testthat/test-derive_param_map.R
tests/testthat/test-derive_param_qtc.R
tests/testthat/test-derive_param_rr.R

I haven’t pushed them yet.
2 questions:

Does the test numbering in the test-*.R files always need to start at 1? For example, in test-derive_param_rr.R, the test headings currently begin at 5 as its original file:

Test 5: new observations are derived correctly ----
Test 6: Message if no new records ----
Should these be renumbered to start at Test 1 and Test 2?

Should I delete the original files to avoid redundancy? These include:
derive_adeg_params.R
derive_advs_params.R
test-derive_adeg_params.R
test-derive_advs_params.R

@bms63 @jeffreyad @bundfussr
I have created the following new files:

R/derive_param_bmi.R
R/derive_param_bsa.R
R/derive_param_map.R
R/derive_param_qtc.R
R/derive_param_rr.R
tests/testthat/test-derive_param_bmi.R
tests/testthat/test-derive_param_bsa.R
tests/testthat/test-derive_param_map.R
tests/testthat/test-derive_param_qtc.R
tests/testthat/test-derive_param_rr.R

I haven’t pushed them yet.
Some questions:

Does the test numbering in the test-*.R files always need to start at 1? For example, in test-derive_param_rr.R, the test headings currently begin at 5 as its original file:

Yes!  If you are restructuring tests or updating old ones it is good to re-number the Tests.  You can install this addin from github pharmaverse4devs to help with renumbering
https://pharmaverse.github.io/admiraldev/dev/articles/unit_test_guidance.html#addin-pharmaverse4devsformat_test_that_file

"## Test 5: new observations are derived correctly ----" "## Test 6: Message if no new records ---- " Should these be renumbered to start at Test 1 and Test 2?

Yes!  Use the addin as it is super quick and easy.  We like our bureaucracy in admiral!

Should I delete the original files to avoid redundancy? These include:
derive_adeg_params.R
derive_advs_params.R
test-derive_adeg_params.R
test-derive_advs_params.R

Yes please remove these files - good to run local devtools::check() but the GHA will also pick up any issues

Should the following R commands be run whenever files are created, deleted, or edited?
styler::style_file()
devtools::document()
roxygen2::roxygenize('.', roclets = c('rd', 'collate', 'namespace'))

The first two - I think the third is needed if multiple people are using different versions of the roxygen and the GHAs checks get confused and recommends it.  Maybe others know why @pharmaverse/admiral


OTG Test Update - RT-5.3

Removed operStatus check on the aggr
Corrected the otg device endpoints for the ipv6 traffic
Fixes #559

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 73.588%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1602

Relevant Lines:
2177

💛  - Coveralls

/gcbrun


data and all_images in fid

Hello, I noticed that data and all_images were used when calculating fid. However, these two images do not seem to be paired. Although all_images looks normal, fid is always around 0.2. What's wrong with this?

Hello,
I think the problem is what I mentioned in this issue.


How to remove instance?

I want to remove the instance of the plugin when i close a modal popup, when i download the canvas image using lc.getImage().toDataURL(), the previous canvas and the new canvas show in the console log, the teardown function give me this error Cannot read property 'willBecomeInactive' of null

Can't debug without a full stack trace or complete example. Reopen if you can provide either.

(I have an idea and will put a null guard in teardown() for the tool in the next release.)


Optional oauth on one point

I have an enpoint that can work both authorized and not authorized, when I don't decorate it with oauth2 I can't seem to be able to get the resource_owner even when I pass the bearer token.

That is true. Resource owner only gets set when an endpoint is authorized.
The scenario where an endpoint can be in both state was not take account for.

i've monkeypatched wine_bouncer 0.3.1 some time ago this way:
module WineBouncer
  class OAuth2 < Grape::Middleware::Base

    #monkeypatch protection behavior. This method shares the given token with the endpoints even if they aren't protected.
    def before
      set_auth_strategy(WineBouncer.configuration.auth_strategy)
      auth_strategy.api_context = context
      #extend the context with auth methods.
      context.extend(WineBouncer::AuthMethods)
      context.protected_endpoint = endpoint_protected?
      self.doorkeeper_request= env # set request for later use.
      doorkeeper_authorize! *auth_scopes if context.protected_endpoint?
      context.doorkeeper_access_token = doorkeeper_token
    end
  end
end

it basically does what you want.
i didn't have time to contribute this as a pull request yet.

This monkey patch works, but would love to see this properly considered within the gem. Have you rethought this @antek-drzewiecki?


Prepare buildpack for automatic dependency updates

We would like to add the dotnet dependency to the pipeline which automatically watches for new versions of a dependency, builds them, and adds them to their corresponding buildpack.
In order for this last step to work smoothly, we need to make sure we're only updating config / yml files instead of Ruby code. For this reason, we've updated the DotnetVersion class to:

Use a compile-extensions script to determine the default dotnet version
Move the SDK version -> Framework version mapping into a new config file, dotnet-versions.yml

There should be no observed change in behavior.
Pivotal Tracker story: https://www.pivotaltracker.com/story/show/130932637
Signed-off-by: Anna Thornton<EMAIL_ADDRESS>

Looks good to me. Definitely an improvement.


Things i found off when i first used it

Description

instead of moving the pixels inside the frame/view its in. movie the view (when holding mouse middle click (read down)

make it so when you change the tolerance value it has effect on your latest selection before you changed the value. Like paint.net

Like paint.net. make it so when you drag and drop an image ontop of another image it asks you if you want to open or layer it

annoying zooming. will try my best to explain it. imagine you are like 10x zoomed in and you want to unzoom just a tiny bit. it will bring your view to the top left corner.

when using a tool and holding any arrow key. it does nothing. try doing this in paint.net and you'll know what i mean

can't scroll the view while holding left click with a tool. for the same purpose as 5.

Additional context
i have only used Pinta for only like 1 minute and these were the problems i had with it instantly.
I'll keep using it and point out other stuff I'd like to be changed
Good luck with the development!
EDIT: it appears that i was using an old version. that is bcs fedora's software manager forced me to download it as a "Fedora linux" package rather than a flatpack. i installed it from flatpack with the terminal and woah. that ui is much better and

is done
isn't

Thanks for the feedback! If you could submit these as separate issues that would be great - it can get very confusing to have multiple bug reports / requests in one issue if they end up being worked on at different times.
For issue 4, this is on the old launchpad bug tracker as https://bugs.launchpad.net/pinta/+bug/1958585 so no need to resubmit - eventually I'll do a migration of those bugs over to GitHub


Resolve UserProfile Loading Errors From Unsecure pages

When we have SSL enabled on single pages we get same-origin errors on loading AdminExperience -> Users -> UserProfile, since it's loaded into an iframe from an unsecure source.
This change will detect this specific case and will force iframe (or popup) source to be loaded as a secure page.

@mean2me for all future Pull Requests can we please get detailed titles that are helpful to all reviewing requests.  References to internal ESW tickets that we do not have exposure to make management of the PR's as well as release notes complicated.  I've updated the title on this one for you.
@ashishpd @daguiler can you communicate this to the rest of the team?

@mitchelsellers  You're right. Actually I forgot to adjust PR title. 🙇 👍

This works as expected now @mean2me. Approving.

@mitchelsellers I added repro steps to the PR's description.


Unable to get issuer cert locally

when using "npm run seed" I get get and error return "UNABLE_TO_GET_ISSUER_CERT_LOCALLY".
I am on Windows 10

seed
node -r dotenv/config ./scripts/seed.js

node:events:496
throw er; // Unhandled 'error' event
^
Error: unable to get local issuer certificate
at TLSSocket.onConnectSecure (node:_tls_wrap:1674:34)
at TLSSocket.emit (node:events:518:28)
at TLSSocket._finishInit (node:_tls_wrap:1085:8)
at ssl.onhandshakedone (node:_tls_wrap:871:12)
Emitted 'error' event on WebSocket instance at:
at emitErrorAndClose (C:\Repo\Github\personal\Next-JS-Dashboard\node_modules\ws\lib\websocket.js:1016:13)
at ClientRequest. (C:\Repo\Github\personal\Next-JS-Dashboard\node_modules\ws\lib\websocket.js:864:5
at ClientRequest.emit (node:events:518:28)
at TLSSocket.socketErrorListener (node:_http_client:495:9)
at TLSSocket.emit (node:events:518:28)
at emitErrorNT (node:internal/streams/destroy:169:8)
at emitErrorCloseNT (node:internal/streams/destroy:128:3)
at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
code: 'UNABLE_TO_GET_ISSUER_CERT_LOCALLY'
}
Node.js v20.11.1

I have the same error,Did you solved it?

Same here!


adds alias for find_by_id in datatore book model

fixes #39

@remi @jeffmendoza PTAL

@remi I used singleton_class.send because it's a class method. PTAL


[Feature Request] :

Pls add Hdtoday.cc extension

Closing as source is a clone of sflix.to, which already exists as an extension.


JSONSchema Validator fails when using format options

Description
Objection v3 returning error unknown format "date" ignored in schema at path "#/properties/tanggalLahir" when using JSON Schema Builder attributes format option
Example
const PersonSchema = SchemaBuilder.emptySchema()
  .addString('id')
  .addString('fulllName')
  .addString('nickname', {}, false)
  .addString('bornDate', { format: 'date' })

Package.json
"objection": "^3.0.0-alpha.4",
"@serafin/schema-builder": "^0.14.2",

Proposed Solution
When I look up at the objection dependencies, i notice it uses the latest version of ajv is ^8. After reading through ajv documentation and issues, it was apparent that ajv version 7 above is implementing breaking changes. They now implement the JSON format specifications in separate module ajv-formats. Looking from the available issues there is solution for this, it was implemented like this
	const AJV = require('ajv').default;
	const addFormats = require('ajv-formats').default;

  	const ajv = new AJV();
  	addFormats(ajv);

I think I will just  modify the AjvValidator.js directly and patch it using patch-package to automate the patching process. Can anyone help me how to implement (adding the ajv formats) this ? Thank you

See this section in the docs https://vincit.github.io/objection.js/api/types/#class-validator

Hello @koskimas thank you for your answer, i have a question how can I override the self.ajv in onCreateAjv function?
\

Why do you need to override it? You can mutate it:
const ajvKeywords = require("ajv-keywords")

...

onCreateAjv(ajv) {
  ajvKeywords(ajv)
}

Weird.. this afternoon i tested it returns an error, then I just tried it again it was successful. Maybe I forgot to extend the BaseModel instead of Model in one of my models before. Thank you @koskimas, and sorry for the inconvenience


Add PR status updates

Fixes: https://github.com/cncf-infra/verify-conformance/issues/16

This appears to be working as intended, including state changes


test_fss failure

https://circleci.com/gh/pipelinedb/pipelinedb/3816

Since we made significant modifications to FSS, going to close this for now. If it happens again, we can re-open the issue.


Blog

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Updating codebase to align with Blog


Add left/right padding to newsbar for long texts / narrow windows

Signed-off-by: Julius Volz<EMAIL_ADDRESS>

@roidelapluie
But will allow myself to merge this as a quick style improvement.