Fault Injection connect question

when i perform fault Injection test，i  failed to connection qemu and reply Boot with module {device} failed!.How can i to solve？

Could you please show me how to reproduce the problem or a screenshot of it to help me locate it?

Thank you for reply.My question is step 4 failed.How can i to resolve this question?
At 2024-11-22 21:50:46, "Ma Zheyu" @.***> wrote:
Could you please show me how to reproduce the problem or a screenshot of it to help me locate it?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

In the fault injection part, I did not generate a success message, and all were crashes

As acknowledged in the paper, PrIntFuzz’s solution does not guarantee 100% success rate, there will be some cases that fail. You can test it on all drivers and then observe the examples in the success folder.

Thank you. We only have 11 successful parts of PCI, most of which are failures, enter a failed emulation device to see that its driver is in use, what is the cause of this situation?

Sorry, I cannot reproduce the problem, everything is fine in my environment. Could you please provide the completed log from scratch so that I can locate the problem? BTW, "in use" means that the device emulation was successful, but it was misclassified.

We have some problems with the fault injection part. When judging the success or not, "Kernel driver in use" cannot be output, but it can be viewed after entering the virtual device. Could you please explain what caused it?

Thank you very much for your reply, but so far I have not found the reason for the error. Let me try again and see if I can solve this problem.

I'm sorry to bother you again, but I still have some questions to ask you.
I can't get the system call list for the interrupt section, and I can't find the 'new_fake_{device_name}.txt' and 'new_qemu_{device_name}.txt' files. How can I get the information about the interrupt section?

Thanks for reporting. This version only included the device emulation code, excluding the interrupt system call descriptions, I'll update the code as soon as you needed.

Thank you. Now when I go to execute the "fuzz.py -s -f" script, output the message "SYZFAIL: Open print_fuzz failed" in the 'syzkaller.out' file. And should I check the final results in the "syzkaller_workdir/crashes report"?

No, you should check why the "printfuzz" module does not work. You can check whether the module is compiled to the kernel and the permission of the module.

I'm sorry to bother you again. I re-did step 7 -"Build the disk image for fuzzing", and I checked for the "print_fuzz.o" file in the "build/linux/linux_fuzz" directory. What should I do to make sure that the "printfuzz" module is included in the kernel? Or how do I check if “printfuzz” is in the kernel?

I'm sorry to bother you again. Could you give us some examples of interruptions, even just one? I would really appreciate it, because it would be very helpful to our work

It looks like the module is compiled in, it could be something else, you need more detailed logs to locate the issue. Please double-check the cause of the error, an error reported by syzkaller is insufficient.

What kind of examples of interruptions do you want? The syzkaller description or something else. BTW, if there are any other questions please start a new issue, it looks clearer, thanks!


Replace Router.events for Next.js 13

related to #304

This is awesome, thanks @catlan !
This is not going to be backwards compatible with Next < v13, right?

import { usePathname, useSearchParams } from "next/navigation"

Are new to Next.js 13, so  it is not backwards compatible.

Not sure if this is related to the changes in this pull request or if the bug exists in current version as well, but for every page view I see two page_view events in the DebugView. I verified that function pageView is only called once per page change. So I assume the gtag is setup up to track page changes as well.

Hi @catlan ! Do you think it could be related to this?
Can you check if upgrading to v2.2.2 solves the issue?

btw @catlan , I'm not feeling very confortable with not making this library compatible with Next < v13, I'm wondering if there could be a way to make it compatible both with >=v13 and <v13. I'm drowning at my job so haven't been able to check this more carefully. I apologize for that, will take the time to come back to this soon.

Not sure if this is related to the changes in this pull request or if the bug exists in current version as well, but for every page view I see two page_view events in the DebugView. I verified that function pageView is only called once per page change. So I assume the gtag is setup up to track page changes as well.

Hi @catlan ! Do you think it could be related to this?
Can you check if upgrading to v2.2.2 solves the issue?

I believe the reason for it is analytics enhanced event measurement. page_view:

An advanced setting on this option controls whether the event is sent based on browser-history events. This measurement option listens for pushState, popState, and replaceState.

The simplest way to keep this change non-breaking would to keep the functionality the same in the usePageViews and create a new hook that implements the next/navigation way. You can run one of them based on a new argument to the  component. Maybe something called appDirectory? This argument defaults to false, so that it does not break current usages, but allows it to be incrementally adopted when used in the app directory.

I can take a crack at finalising this PR this week if wanted?
@Gawdfrey That would be awesome! And I like your proposal for making it a new opt-in hook so people can migrate when they desire without breaking current functionality.


Allow "..." to be used in resolve.extensions to extend the default values

Feature request
Allow "..." to be used in resolve.extensions to extend the default values.
What is the expected behavior?
resolve: {
    extensions: ["...", ".ts", ".vue"] // Allow this.
}

What is motivation or use case for adding/changing the behavior?
Extend the defaults without having to redefine them.
How should this be implemented in your opinion?
Similar to how it's implemented in other places, I assume.
Are you willing to work on this yourself?
Not in the short term.

Ya, this is a problem in Vuetify that I reported here. I'll create a separate issue for it soon.

'...' does work as expected in my test, here's an example https://codesandbox.io/s/webpack-forked-13d1z?file=/webpack.config.js.
Notice that we should inspect resolve.byDependency instead of resolve.extensions for the default value, the latter defaults to [], i.e., that default value listed in documentation for resolve.extensions is wrong.

@chenxsan,
Yes, it does work. I thought that it wasn't working because .mjs was not being included, but it was the documentation that was incorrect (hence this issue).


PipeClient::read sometimes causes panic

The panic originates from lib.rs:974:26 (PipeClient::drop) eventually tracing back to lib.rs:1036 (the end of ReadHandle::wait). The message reads CANCEL ERROR Os { code: 1168, kind: Other, message: "Element not found." }.
It occurs when reading with a zero timeout, but only sometimes. My guess is a race condition.
Code sample
use std::io;
use std::io::{BufRead, BufReader, ErrorKind, Read};
use std::time::Duration;

fn main() -> io::Result<()> {
    let mut client = named_pipe::PipeClient::connect(r"\\.\pipe\abcd")?;
    client.set_read_timeout(Some(Duration::from_millis(0)));

    let mut reader = BufReader::new(client);
    loop {
        while let Some(line) = read_line(&mut reader)? {
            print!("read line: {}", line);
        }

        // do some unrelated work
    }
}

fn read_line<T: Read>(reader: &mut BufReader<T>) -> io::Result<Option<String>> {
    let mut line = String::new();
    match reader.read_line(&mut line) {
        Ok(_) => Ok(Some(line)),
        Err(ref err) if err.kind() == ErrorKind::TimedOut => Ok(None),
        Err(err) => Err(err),
    }
}

Hi and thanks for report. I'll look into it.

Fixed in v0.4.1


switch from ujson to orjson

I got an email some time ago from @ijl:

removed (I was asked to remove the content of the email)

(I asked if they could create an issue but I guess it got forgotten)
If we want to move to orjson (and it sounds to me like a pretty good idea) it should be done before v1.
I guess we should make orjson a requirement and use it always rather than switch between ujson and json.
Any feedback on this?
I guess it would allow us to simplify pydantic_encoder eg. by no longer having to deal with datetime objects.
However there would also be some breaking changes I think.
@tiangolo do you have any view on this? I know you have a significant requirement for serialising to JSON but also to "JSON-like python objects". Does orjson help or hinder this?

Thanks for tagging me!
I received a very similar email suggesting it for FastAPI, and the PR never arrived as well :joy:
I was thinking about writing a PR to try it out here and see what would happen but never found the time.
I don't think this would affect my use cases, or if anything, would improve them. But I haven't checked thoroughly.

I think the use in both projects would be rather involved. If the goal is to remove ujson, I have a minimal patch in #599.

replaced by #714.


Improve importHref()

uses altering test assets to ensure tests do not affect each other
ensure importHref always returns a link element
ensure onload() and onerror() functions are only called once when the same element was loaded again

Blocked by https://github.com/webcomponents/webcomponentsjs/pull/655 because the load event is invoked twice when a failed import attempt is repeated.
duplicate of #4209 that points against master instead of the branch behind #4200 (refactor-import-href)

These changes have been incorporated in #4305. Please republish this PR if you find any changes we missed there.

Wow, thanks. I didn't even have to rebase 🎂


Marketplace assistant

Added api name and version in the payload history

 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.


fix: add config partials for dynamic frontend plugins

Fixes: https://github.com/janus-idp/backstage-plugins/issues/960

Thanks @davidfestal , all should be addressed now 🙂

/approve
/lgtm


[feature] dont create a new revision if current document data is not different from the last revision

atm even if the file have no difference between it and the last time it was saved, the ext still create a new revision, which create unnecessary noise in the history list.

seeing this too, on Mac.  I save a file, and immediately it is marked dirty (modified) again even though the content didn't change

https://marketplace.visualstudio.com/items?itemName=ctf0.local-history-new


[FLINK-5984] [table] add resetAccumulator method for AggregateFunction

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[x] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[ ] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[x] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

@fhueske , do you have plan to merge FLINK-5983 and FLINK-5963 very soon? This PR probably need rebase once your two PRs are merged, as changes are overlapped.

Thanks @shaoxuan-wang. The PR looks good.
FLINK-5983 has been merged and FLINK-5963 will be merged soon.
It would be great if you could rebase your code on the master once that was done.
Thanks, Fabian

@fhueske and @shaoxuan-wang. I will rebase and merge this.


bug(global): White screen before loading into app

Issue:

If you open a new tab and go to localhost it flashes a white screen for a split second and then loads into the app.

This is messing up the playwright tests, when a valid response is done in the browser, playwright thinks everything is then ready to test but it isn't due to the split second it shows the white screen

Steps to reproduce:

Open a new tab
Go to Localhost

*To replicate this on iOS build the app and open it. Usually opening on the first time after building will show the bug
Devices:

iOS
Desktop

Screen recording:
https://github.com/user-attachments/assets/2345cf5c-8536-4b19-85be-ede24905a7a2
https://github.com/user-attachments/assets/a09b1896-24c0-493b-bbcb-94fbb14771ab

closing this since a flash for a white screen for a split second and then loads into the app is not a priority atm


[SPARK-15243][ML][SQL][PYSPARK] Param methods should use basestring for type checking

What changes were proposed in this pull request?
The following methods used isinstance(value, str) for checking string types in Python ML params:

_resolveParam(param)
hasParam(param)

This causes a ValueError in Python 2.x when param is a unicode string:
>>> from pyspark.ml.classification import LogisticRegression
>>> lr = LogisticRegression()
>>> lr.hasParam("threshold")
True
>>> lr.hasParam(u"threshold")
Traceback (most recent call last):
 ...
    raise TypeError("hasParam(): paramName must be a string")
TypeError: hasParam(): paramName must be a string
>>>

How was this patch tested?
Unit tests added to python/ml/tests.py

Test build #58289 has started for PR 13036 at commit 878fc5f.

Test build #58289 has finished for PR 13036 at commit 878fc5f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58289/
Test PASSed.

Currently investigating some other usages of isinstance(obj, str) for this PR. Will update soon.

Test build #58386 has started for PR 13036 at commit b04ac41.

I updated instances of similar checks in the sql library as noted on the Jira. I searched and this type of check now only exists here and here. They don't cause problems with unicode though, so I did not change them, but I can do that if needed.
cc @viirya I'm not as familiar with the sql library, could you check those changes?
Also cc @holdenk @davies

Test build #58386 has finished for PR 13036 at commit b04ac41.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58386/
Test PASSed.

lgtm

Checking old PRs---is this active still?  The ML parts look good to me.  I haven't checked the SQL ones carefully.

Just a quick ping @sethah - I know your pretty busy but I'm assuming this is still active. One minor note is it seems there is another new addition to types.py which maybe should also be changed.

Test build #66542 has started for PR 13036 at commit 48f0557.

Test build #66542 has finished for PR 13036 at commit 48f0557.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66542/
Test PASSed.

Merged build finished. Test PASSed.

Test build #67032 has started for PR 13036 at commit c6a8828.

Test build #67032 has finished for PR 13036 at commit c6a8828.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67032/
Test PASSed.

Merged build finished. Test PASSed.

ping @holdenk @viirya
I think this is ready now :)

Test build #67093 has started for PR 13036 at commit 976d682.

Test build #67093 has finished for PR 13036 at commit 976d682.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67093/
Test PASSed.

Gentle ping, whats the status of this PR?

@holdenk please feel free to take this over. Can't find time to work on it

Ok, lets see if maybe @zero323 or @HyukjinKwon are interested in taking this over. Otherwise I'll add this to my backlog.

I am happy to do so. I assume that It seems already almost done except for https://github.com/apache/spark/pull/13036#discussion_r84476560?


Event classes should have a public constructor (or not be final)

Event classes (e.g. com.github.messenger4j.receive.events.OptInEvent) is final and has only a private constructor.
This makes testing impossible. For example I have a class that implements com.github.messenger4j.receive.handlers.OptInEventHandler but I cannot write a junit test for it because I cannot easily create an instance of com.github.messenger4j.receive.events.OptInEvent in my test.

Thanks for your valuable feedback!
You are right that the current implementation makes unit testing hard. That should be changed.
I think it's a good solution to make the private constructors of the event classes public (be aware of inner classes that have to be changed too, e.g. Attachment in AttachmentMessageEvent).
But the event classes should be final. I think there is no reason to derive classes from the existing event classes, because they implement / are part of the spec.
What do you think?

PR accepted - thanks Jan for your contribution.


add deserialize_ignore_any

Fixes https://github.com/jonasbb/serde_with/issues/250

Thanks for updating the description. Once CI passes you can merge the PR by writing bors r+ in a comment.
bors delegate=lovasoa

Thanks for updating the description. Once CI passes you can merge the PR by writing bors r+ in a comment.
bors delegate=lovasoa

bors r+

bors r+


How to print an image ?

First of all thanks for the project. it works as expected.
But i need to print an image. Despite i convert the image to byte , i could not do it.
How should i convert the image to byte? thanks.

if you want to print a image you need to use ESC/POS command
Check your printer's corresponding Print image Commands


BCD Denoiser clamps output [$15]

https://forums.luxcorerender.org/viewtopic.php?f=4&t=385

There is a $15 open bounty on this issue. Add to the bounty at Bountysource.

The problem should have been fixed by last commit.


Docs: Update README and other documentation

@luise, could you rebase?  github is confused.  Alternatively you could close and reopen the PR

Yup, I'll submit another PR soon.


Prefer the new name of proto_helpers

Twisted deprecated the twisted.test.proto_helpers module and introduced a new twisted.internet.testing module with all of the same contents.

Thanks!


typescript: accept corpus argument [phab:D1545]

Created by evmar at 2017-04-19 22:04:24:
The exposed indexing function now accepts a 'corpus' argument,
which is then used in produced VNames.

fromberger accepted this revision at 2017-04-19 22:13:26

fromberger closed via commit 81927a7e5cf4b0264c1a6e8732ceb5ec285c4c1e at 2017-04-19 22:13:50


Fixed #30020 -- Fixed reading nulls with LayerMapping.

Overview
Ticket 30020.
Added a GDAL function wrapper to recognize if a field is either null or "unset".
Modified LayerMapping importer to import null or unset GDAL values as null. Previously, null numeric values in an input file would be set as zeroes on the destination model, and null strings would be set as empty strings.
Testing notes
I've run the gis_tests suite successfully against PostGIS, Spatialite, and MySQL database backends. For each, tests succeeded with both python 2.7 / GDAL 2.2, and with python 3.6 / GDAL 2.1.
I made a small demo app here. The demo app is in Django 1.11, and can run its tests against a separate 1.11 backport branch with the commits here cherry-picked.
The demo app repository can also run the core Django tests with Docker containers; there are branches for the different database backends. I used it to run the core GIS suite of tests against the Django master-based branch here.

Do you think it would be possible to replace the Shapefile test data with a GeoJSON one? Or is the null issue really specific to shapefiles? Having a text instead of binary data source would be nice for readability.

Wow, great! Existing tests are based on Shapefile probably for historical reasons, I guess GeoJSON layers were not supported in the past.
Now do you think it's worth keeping the Shapefile tests in this patch? In other words, is anything in the patch specific to Shapefiles?

By the way, feel free to add your name to the AUTHORS file.

I removed the test Shapefile. Switching the gdal_tests/test_ds.py data source added here to use the GeoJSON driver instead of the Shapefile driver surfaced an unrelated issue with reading numbers from GeoJSON, in which OFTReal typed fields were treated as OFTInteger because they had no precision set.
I removed the logic to cast reals without precision to integers in 2b9faa0, which fixed the issue and did not break any existing tests. It's unclear why that logic was in place, as there is no comment to explain the choice, and the commit history that adds it predates the repository move from SVN to git.

Good progress, thanks!
@timgraham or @carltongibson, it may be time for another review. Would you cut the changes in more than one commit?

Sorry, but I'm not sure what "cut the changes in more than one commit" means. Would you like me to rebase down to one commit?

Good progress, thanks!
@timgraham or @carltongibson, it may be time for another review. Would you cut the changes in more than one commit?

Rebasing to one commit isn't the issue, it can easily be done before the merge. However, one might argue that some changes in your patch should be committed separately. That's generally asked by the Django fellows at the final review stage, let's wait for that to happen.

Kathryn, may I ask you to rebase your branch (after 0004daa53689)?
I think you may also squash to one commit. If you are asked later to separate commits, it will not match the current series anyway.

Some failures on Ubuntu 16.04 (possibly due to GDAL 1.11 which we may drop support for in Django 3.0 as GDAL 1.11 was released over 5 years ago, but later versions of GDAL may also be affected):
======================================================================
ERROR: test_null_number_imported (gis_tests.layermap.tests.LayerMapTest)
LayerMapping import of GeoJSON with a null numeric value.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/media/sf_django/tests/gis_tests/layermap/tests.py", line 329, in test_null_number_imported
    lm = LayerMapping(HasNulls, has_nulls_geojson, has_nulls_mapping)
  File "/media/sf_django/django/contrib/gis/utils/layermapping.py", line 108, in __init__
    self.check_layer()
  File "/media/sf_django/django/contrib/gis/utils/layermapping.py", line 258, in check_layer
    (ogr_field, ogr_field.__name__, fld_name))
django.contrib.gis.utils.layermapping.LayerMapError: OGR field "<class 'django.contrib.gis.gdal.field.OFTString'>" (of type OFTString) cannot be mapped to Django DateTimeField.

======================================================================
ERROR: test_null_number_imported_not_allowed (gis_tests.layermap.tests.LayerMapTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/media/sf_django/tests/gis_tests/layermap/tests.py", line 371, in test_null_number_imported_not_allowed
    lm = LayerMapping(DoesNotAllowNulls, has_nulls_geojson, has_nulls_mapping)
  File "/media/sf_django/django/contrib/gis/utils/layermapping.py", line 108, in __init__
    self.check_layer()
  File "/media/sf_django/django/contrib/gis/utils/layermapping.py", line 258, in check_layer
    (ogr_field, ogr_field.__name__, fld_name))
django.contrib.gis.utils.layermapping.LayerMapError: OGR field "<class 'django.contrib.gis.gdal.field.OFTString'>" (of type OFTString) cannot be mapped to Django DateTimeField.

======================================================================
ERROR: test_null_string_imported (gis_tests.layermap.tests.LayerMapTest)
Test LayerMapping import of GeoJSON with a null string value.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/media/sf_django/tests/gis_tests/layermap/tests.py", line 337, in test_null_string_imported
    lm = LayerMapping(HasNulls, has_nulls_geojson, has_nulls_mapping)
  File "/media/sf_django/django/contrib/gis/utils/layermapping.py", line 108, in __init__
    self.check_layer()
  File "/media/sf_django/django/contrib/gis/utils/layermapping.py", line 258, in check_layer
    (ogr_field, ogr_field.__name__, fld_name))
django.contrib.gis.utils.layermapping.LayerMapError: OGR field "<class 'django.contrib.gis.gdal.field.OFTString'>" (of type OFTString) cannot be mapped to Django DateTimeField.

======================================================================
ERROR: test_nullable_boolean_imported (gis_tests.layermap.tests.LayerMapTest)
LayerMapping import of GeoJSON with a nullable boolean value.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/media/sf_django/tests/gis_tests/layermap/tests.py", line 346, in test_nullable_boolean_imported
    lm = LayerMapping(HasNulls, has_nulls_geojson, has_nulls_mapping)
  File "/media/sf_django/django/contrib/gis/utils/layermapping.py", line 108, in __init__
    self.check_layer()
  File "/media/sf_django/django/contrib/gis/utils/layermapping.py", line 258, in check_layer
    (ogr_field, ogr_field.__name__, fld_name))
django.contrib.gis.utils.layermapping.LayerMapError: OGR field "<class 'django.contrib.gis.gdal.field.OFTString'>" (of type OFTString) cannot be mapped to Django DateTimeField.

======================================================================
ERROR: test_nullable_datetime_imported (gis_tests.layermap.tests.LayerMapTest)
LayerMapping import of GeoJSON with a nullable date/time value.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/media/sf_django/tests/gis_tests/layermap/tests.py", line 354, in test_nullable_datetime_imported
    lm = LayerMapping(HasNulls, has_nulls_geojson, has_nulls_mapping)
  File "/media/sf_django/django/contrib/gis/utils/layermapping.py", line 108, in __init__
    self.check_layer()
  File "/media/sf_django/django/contrib/gis/utils/layermapping.py", line 258, in check_layer
    (ogr_field, ogr_field.__name__, fld_name))
django.contrib.gis.utils.layermapping.LayerMapError: OGR field "<class 'django.contrib.gis.gdal.field.OFTString'>" (of type OFTString) cannot be mapped to Django DateTimeField.

======================================================================
ERROR: test_uuids_imported (gis_tests.layermap.tests.LayerMapTest)
LayerMapping import of GeoJSON with UUIDs.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/media/sf_django/tests/gis_tests/layermap/tests.py", line 362, in test_uuids_imported
    lm = LayerMapping(HasNulls, has_nulls_geojson, has_nulls_mapping)
  File "/media/sf_django/django/contrib/gis/utils/layermapping.py", line 108, in __init__
    self.check_layer()
  File "/media/sf_django/django/contrib/gis/utils/layermapping.py", line 258, in check_layer
    (ogr_field, ogr_field.__name__, fld_name))
django.contrib.gis.utils.layermapping.LayerMapError: OGR field "<class 'django.contrib.gis.gdal.field.OFTString'>" (of type OFTString) cannot be mapped to Django DateTimeField.

======================================================================
FAIL: test03a_layers (gis_tests.gdal_tests.test_ds.DataSourceTest)
Testing Data Source Layers.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/media/sf_django/tests/gis_tests/gdal_tests/test_ds.py", line 164, in test03a_layers
    self.assertEqual(fld_value, layer.get_fields(fld_name))
AssertionError: Lists differ: [5, None, 8] != ['5', None, '8']

First differing element 0:
5
'5'

- [5, None, 8]
+ ['5', None, '8']
?  + +        + +

======================================================================
FAIL: test04_features (gis_tests.gdal_tests.test_ds.DataSourceTest)
Testing Data Source Features.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/media/sf_django/tests/gis_tests/gdal_tests/test_ds.py", line 237, in test04_features
    self.assertIsInstance(feat[k], v)
AssertionError: <django.contrib.gis.gdal.field.OFTReal object at 0x7f459af445f8> is not an instance of <class 'django.contrib.gis.gdal.field.OFTInteger'>

It looks like those were failures with the existing date/time field mapping, which was not added here, but I did add tests for it here; there were no tests for date/time field mappings in GDAL/LayerMapping previously, as far as I could tell. I'l try to reproduce this locally.

I confirmed that the failures aren't present with GDAL 2.0 so I think we can simply drop support for GDAL 1.11 (https://github.com/django/django/pull/10959).

I was able to reproduce the test failures with GDAL 1.11. The DateTime failures are due to the GDAL GeoJSON driver not adding support for DateTime until version 2.0, and so can be addressed with GDAL version checks on the tests.
I have a branch here that addresses most of the test failures, but not all. There seems to be an issue with GDAL 1.11 integer/real type identification in GeoJSON I haven't been able to pin down.


Add the delete feature and a new match function in webstep file

I added the delete feature

Codecov Report

Merging #101 (edb341d) into master (0dbe75c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #101   +/-   ##
=======================================
  Coverage   95.70%   95.70%
=======================================
  Files           7        7
  Lines         349      349
=======================================
  Hits          334      334
  Misses         15       15

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


EthicalAd: don't force "Coming soon" ad and show paid ads

We were testing the ad positioning and telling our users that EthicalAd is coming. Now that we already tested this and inform our users, it seems we are ready to deploy the ads to the documentation.

@ericholscher I think we want to deploy this tomorrow, right?

Don't worry. We can release it today quickly. Let me know.


Require the angle bracket when not at top level

Fixes #6
This will require further testing because I haven't explored which other implications this change has.

return [
'models' => [

    /*
     * When using the "HasRoles" trait from this package, we need to know which
     * Eloquent model should be used to retrieve your permissions. Of course, it
     * is often just the "Permission" model but you may use whatever you like.
     *
     * The model you want to use as a Permission model needs to implement the
     * `Spatie\Permission\Contracts\Permission` contract.
     */

    'permission' => Spatie\Permission\Models\Permission::class,

    /*
     * When using the "HasRoles" trait from this package, we need to know which
     * Eloquent model should be used to retrieve your roles. Of course, it
     * is often just the "Role" model but you may use whatever you like.
     *
     * The model you want to use as a Role model needs to implement the
     * `Spatie\Permission\Contracts\Role` contract.
     */

    'role' => Spatie\Permission\Models\Role::class,

],

'table_names' => [

    /*
     * When using the "HasRoles" trait from this package, we need to know which
     * table should be used to retrieve your roles. We have chosen a basic
     * default value but you may easily change it to any table you like.
     */

    'roles' => 'roles',

    /*
     * When using the "HasRoles" trait from this package, we need to know which
     * table should be used to retrieve your permissions. We have chosen a basic
     * default value but you may easily change it to any table you like.
     */

    'permissions' => 'permissions',

    /*
     * When using the "HasRoles" trait from this package, we need to know which
     * table should be used to retrieve your models permissions. We have chosen a
     * basic default value but you may easily change it to any table you like.
     */

    'model_has_permissions' => 'model_has_permissions',

    /*
     * When using the "HasRoles" trait from this package, we need to know which
     * table should be used to retrieve your models roles. We have chosen a
     * basic default value but you may easily change it to any table you like.
     */

    'model_has_roles' => 'model_has_roles',

    /*
     * When using the "HasRoles" trait from this package, we need to know which
     * table should be used to retrieve your roles permissions. We have chosen a
     * basic default value but you may easily change it to any table you like.
     */

    'role_has_permissions' => 'role_has_permissions',
],

'column_names' => [

    /*
     * Change this if you want to name the related model primary key other than
     * `model_id`.
     *
     * For example, this would be nice if your primary keys are all UUIDs. In
     * that case, name this `model_uuid`.
     */
    'model_morph_key' => 'model_id',
],

/*
 * By default all permissions will be cached for 24 hours unless a permission or
 * role is updated. Then the cache will be flushed immediately.
 */

'cache_expiration_time' => 60 * 24,

/*
 * When set to true, the required permission/role names are added to the exception
 * message. This could be considered an information leak in some contexts, so
 * the default setting is false here for optimum safety.
 */

'display_permission_in_exception' => false,

];


speakers page

While we do not have any speakers yet, we should have the page ready to go.
Info for the page will include:
[] Name
[] Session Name
[] Session Info
[] Image
[] Link to singular speaker page with extra details about the speaker
[] Twitter name

Gonna give this one a go

closed vie #35


Search Bar Not working.

Describe the bug
The search bar is not working , it should search the project when user enters a project name he/she wants...
Expected behavior
it should search the project when user enters a project name he/she wants...
Screenshots

bro press enter or click on the map icon , its working

Not in that way brother ,I want that  if someone searches for a project by typing it's name in it the project should show up . In this way I want this search bar to work.

Please assign to me this issue

Not in that way brother ,I want that if someone searches for a project by typing it's name in it the project should show up . In this way I want this search bar to work.

ohh :)


Hhh 11090 master

Pull request for HHH-11090.

You asked for me to comment on the general approach...  So the (big!) part missing here is supporting changes in these unique keys.  This is something specific I had to account for (for example) in natural-id support.  This is the code in org.hibernate.engine.internal.NaturalIdXrefDelegate.  You'd need something similar here.  IMO this overhead is not worth it.

Thanks for the feedback.
I just came across the code you mention for natural IDs. I will look into whether that functionality can be leveraged instead.

If the UK is the natural-id it certainly could.

Well actually stepping back on this a bit...
I guess your work here so far is really just to add an entry into StatefulPersistenceContext#entitiesByUniqueKey in places where we are missing it.  Since we already cache these values, I think its appropriate to push those values properly.  That is fine.  The lack of invalidation on change is a separate issue, since we already do not do that

@sebersole, for mutable, unique properties that are not natural IDs, I think it makes sense to simply delete the mappings in StatefulPersistenceContext#entitiesByUniqueKey after the (load, merge, etc) operation completes. It could be in the OperationContext in H6. I'd have to look into how this could work in H5. WDYT?

That is a safe approach for sure.  And if we are moving to that paradigm of scoping that cache to the context of that load, then I think it makes sense to move that off of StatefulPersistenceContext.  I'm not sure if OperationContext is the correct place for that, or something more like org.hibernate.engine.loading.internal.LoadContexts, or something entirely new.  LoadContexts is available already, so that would not necessarily have to wait for 6.0.
Let's hold off on the 6.0 part until after we get OperationalContext integrated into that base and then you, me, @dreab8 and @Naros can all discuss that aspect more informed.

@gbadner Where do we stand with this one?


nodeos consumes 100% cpu

nodeos v1.8.12 consumes nearly 100% cpu when syncing with other nodes.
Is there any way to improve this?

Improve in what way? You want an option to sync slower?


Updated the urls to lib

Hi @madhugb, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

I’m sorry, we can’t apply this PR to our repo, because it breaks webpack dev server. You can use standalone.html instead of index.html.


parsing options and serializing arrays

several parsing options are added. they are organized in classes because there are many of them. a "text" based API to configure options is provided.
another feature is the ability to serialize a column of arrays. the array is "unnested" and column names to use are supplied by user. this is useful for writing out csv after doing transforms on the data that "expand" the number of columns e.g. one hot encode a category. this can be improved later. e.g. sparse vector from mllib can replace the array.

Please update it to rebase against master for tests to run. Also please revert changes to build.sbt.

Current coverage is 84.79%

Merging #113 into master will decrease coverage by -0.22% as of 6c0dea6

@@            master    #113   diff @@
======================================
  Files           10      11     +1
  Stmts          407     526   +119
  Branches       125     150    +25
  Methods          0       0
======================================
+ Hit            346     446   +100
  Partial          0       0
- Missed          61      80    +19

Review entire Coverage Diff

Powered by Codecov

@falaki should be ready to merge now.

@mohitjaggi This is fairly large. I am about to publish a release with schema inference and all the recent improvements, and then I will review this.

@mohitjaggi this is packing too much into a single PR. Would you please split it. Please first submit one for number parsing improvements and another for arrays. On arrays within CSV it would be good to post an issue and gather some feedback from community first.

will do
On Thu, Aug 13, 2015 at 10:14 AM, Hossein Falaki<EMAIL_ADDRESS>wrote:

@mohitjaggi https://github.com/mohitjaggi this is packing too much into
a single PR. Would you please split it. Please first submit one for number
parsing improvements and another for arrays. On arrays within CSV it would
be good to post an issue and gather some feedback from community first.
—
Reply to this email directly or view it on GitHub
https://github.com/databricks/spark-csv/pull/113#issuecomment-130765125.

see #124

Wouldn't this make sense to close this if #124 is subset from this and you are willing to make some more?


Add checkstyle to projects and fix some errors

Since you said you wanted Checkstyle in #195, I thought I'd add it.
The CI should fail as there are still some checkstyle errors. I did resolve some  errors. Though Line longer than 100 characters, Comment matches // TOOD: and More than 7 parameters, I did not since I don't know how you'd like to handle those errors.
The checkstyle.xml is from your ButterKnife Project.

What has more than 7 params?

ViewLayoutChangeEvent.java:16
ViewLayoutChangeEvent.java:24
ViewLayoutChangeEventOnSubscribe.java:22
ViewLayoutChangeOnSubscribe.java:21

Ah, forgot those even existed. I'll have a look tomorrow!

What's also possible is to suppress some warnings using @SuppressWarnings("checkstyle:<rule>"). Though as far as I see the checkstyle.xml file would need to be configured for that.


cmake: use project source paths for includes

replaced CMAKE_SOURCE_DIR with PROJECT_SOURCE_DIR
Issue:
Using gt project as submodule in super project with add_subdirectory
During compilation base and gadget includes not found because CMAKE_SOURCE_DIR points on super project directory, not gt
Solution:
Use either PROJECT_SOURCE_DIR or gt_SOURCE_DIR which is the same (ref here).
I replace to PROJECT_SOURCE_DIR because if PROJECT(gt) will changed somehow, it will not affect other scripts.

gt shouldn't really be used as a submodule, it is a project in its own
right and should be used as a dependency instead.
So can you say some more about what situation you are using gt in,
which project is using it as a submodule, which packaging system you
are using in your situation and why you are using gt as a submodule
instead of a dependency?
The patch doesn't break usage as a separate project/dependency though,
so it should be OK to include it after some more info is given.
PS: this is a better link for the PROJECT_SOURCE_DIR variable:
https://cmake.org/cmake/help/latest/variable/PROJECT_SOURCE_DIR.html
--
bye,
pabs
https://bonedaddy.net/pabs3/

gt shouldn't really be used as a submodule, it is a project in its own right and should be used as a dependency instead.

IMO, if you're using CMake you should flawlessly include it as you wish.

So can you say some more about what situation you are using gt in, which project is using it as a submodule, which packaging system you are using in your situation and why you are using gt as a submodule instead of a dependency?

Situation is quite simple - I want to try gt tool in buildroot build for embedded target, but it's not present as package.
I don't want to modify buildroot at that time (but I can, as well as use patched version of gt), because I'm not sure if I will use it later.
It depends on evaluation results.

The patch doesn't break usage as a separate project/dependency though, so it should be OK to include it after some more info is given.

Because it's correct usage for CMake project keyword.

PS: this is a better link for the PROJECT_SOURCE_DIR variable: https://cmake.org/cmake/help/latest/variable/PROJECT_SOURCE_DIR.html
…

Thank you for the link. I know where latest CMake documentation located. But you using cmake_minimum_required(VERSION 2.8) so I point to right location for version your using.

-- bye, pabs https://bonedaddy.net/pabs3/

Thank you.

Thanks for the info.
I have merged the patch, making two changes:
I have reworded the commit message to include more detail.
I have dropped the whitespace change, it is best to do separate logical
changes in separate commits.
Please do try to get gt into buildroot, it will be useful for others to
have it even if you don't end up using it.
--
bye,
pabs
https://bonedaddy.net/pabs3/


Go to symbol for AppleScript ?

One feature missing in Visual Studio Code for AppleScript files is the ability to go to a symbol. It would be useful to have this feature to navigate in long scripts, from a function to the next or to jump to a specific function in the file.
I don't know if it is possible, but if that's the case, it would be a welcome addition I think.

I'm sure it's possible, but I'm not sure how. The syntax definition is taken from the AppleScript TextMate bundle.

OK great to know it's possible. I'm not sure I will be able to help either though…

@idleberg it would be so helpful to have this feature! I think sublime has this. Thank you for all the work 👍

@idleberg any luck on this?


Hyperlinks Not Working Outside Chromium

Using latest version of the tool, sync works fine, however I can't get links outside of Chromium to open. When I use spotlight search or click links in other applications (like Outlook) nothing happens.
Looks like according to the commit history this should be working, but I'm not sure how to troubleshoot this.

get me output of
$cat /Applications/Chromium.app/Contents/MacOS/Chromium

reopen if you want to update


docs: Add related resources to readme

:tada: This issue has been resolved in version 0.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Empty file upload warning and flash_message rewrite

Warn students if they upload an empty file
Update to the flash_message function so that it can render partial html pages as well as just strings.
Update to the submission_controller so that it now uses the assignment_content layout

Also I noticed some tests fail due to an annoying reason. I recommend thinking about how you want to fix this and running it by me before you just change all the tests (which is a pain).

@david-yz-liu .  I also noticed that there is an update_files method in the assignment_controller as well (though it looks like it isn't currently used). Should I update this method to mirror the changes that have been made to the update_files method in the submissions_controller? Should this method be removed?

@david-yz-liu
My suggestion for handling the failing tests on the flash message content is to strip off all html content.
For example, a line in a test currently looks like:
expect([t('criterion_deleted_success')]).to eql(flash[:success])

so we change it to:
expect(t('criterion_deleted_success')).to eql(Nokogiri::HTML(flash[:success][0]).text )

or even:
expect(Nokogiri::HTML(t('criterion_deleted_success')).text).to eql(Nokogiri::HTML(flash[:success][0]).text )

if we expect there to be tags in some of our internationalized strings.
We could even add some other string normalization steps after stripping the tags just in case we change the spacing (for example) when adding tags to the flash messages.  In either case I would probably like to create a helper function that we can call from the tests just to keep the test code clean looking.

@mishaschwartz your proposal for changing the tests looks good to me.


[stdlib] Add contiguous storage forwarding to AnySequence, etc

Means this now works:
(swift) [2,3].withContiguousStorageIfAvailable { print($0) }
UnsafeBufferPointer(start: 0x00007fa99cc6de20, count: 2)
// r0 : ()? = Optional(())
(swift) AnySequence([2,3]).withContiguousStorageIfAvailable { print($0) }
UnsafeBufferPointer(start: 0x00007fa99cc7e060, count: 2)
// r1 : ()? = Optional(())

Some more low-hanging fruit. Might as well add to this one rather than making a new PR. Documentation edits are in the style of forEach - "Calls the given closure ...".
I don't expect benchmark improvements because we don't actually use this function in the standard library (yet). We should do, and it should improve Slice -> Array performance (Slice doesn't implement _copyContents/_copyToContiguousArray AFAICT).

@airspeedswift is this okay to merge?

Ping @airspeedswift

@swift-ci please test

These additions will need availability markup. See this post for details on how.

Could you add tests exercising the new functionality?

@karwa Just pinging the old issues.  Could you give us a rebase here and address the review feedback?

Can one of the admins verify this patch?

Please update the base branch to main by Oct 5th otherwise the pull request will be closed automatically.

How to change the base branch: (Link)
More detail about the branch update: (Link)


Refer to top button as "Buzz!" instead of "Red"

The top button is officially referred to as the Buzz! button, as on real buzzers, the logo is engraved/embossed/whatever into the button. (This is also how it's referred to in-game as well as manuals - I verified by checking my Mega Quiz manual and it calls this button the Buzz! button)
I've changed COLOUR to return "Buzz!" instead of "Red" to reflect this, so when it's shown to the user it'll have the proper name.

Hi, thanks for your contribution! Sorry for not having noticed this pull request before now.
I don't want to change a constant at this stage for fear of breaking anyone's implementation.
Instead, I have added an additional BUTTONLABEL dict which works as you suggest for anyone who would like button names rather than colours. See most recent commit.


Proceed Button disappears when clicking on Shipping Address checkbox  in Checkout. Shipping Details should not appear for Virtual and Downloadable product.

Is your feature request related to a problem? Please describe.
Bagisto Version - V2.1.0
Pre-Conditions

Guest User/Logged in User checkout process.
Add virtual product or downloadable product (for logged in customer)

Description
When we checkout a Virtual or Downloadable product, there when we selecte the checkbox for the shipping method. We can see that the Proceed button disappears.
Actual Result

Proceed button disappears hence we are unable to select the payment method.
After unselecting the shipping check box we can see the payment options and then we can checkout.

Video for reference
https://github.com/bagisto/bagisto/assets/92721837/904a3314-ac45-4b90-9039-0b07f902ae5e
Describe the solution you'd like
Expected Result
For Products like Virtual and Downloadable, we should not show the shipping related concern.
If in case we are showing shipping address then Proceed button should not hide.
Describe alternatives you've considered
No response
Additional context
No response

Already fixed in #9547


Fixing the docker argument in case of pipeline start

This should close #292.

/test

/test

/test

Codecov Report

Merging #294 (2d40c7f) into master (da4aac4) will decrease coverage by 0.01%.
The diff coverage is 81.25%.

@@            Coverage Diff             @@
##           master     #294      +/-   ##
==========================================
- Coverage   58.21%   58.20%   -0.02%
==========================================
  Files          58       58
  Lines        4186     4180       -6
==========================================
- Hits         2437     2433       -4
  Misses       1284     1284
+ Partials      465      463       -2

Impacted Files
Coverage Δ

providers/pipelines/pipeline.go
48.37% <0.00%> (-0.64%)
:arrow_down:

workers/scheduler/gaiascheduler/scheduler.go
55.97% <91.66%> (+0.13%)
:arrow_up:

handlers/vault.go
55.17% <100.00%> (ø)

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 da4aac4...2d40c7f. Read the comment docs.

/test


Compare substance to alternatives

Hi,
I'm in the process of choosing one of the following frameworks: substance, quill, prosemirror, slate.
It would be nice to have in the home page a comparative between Substance and alternatives.  I would like to know what exactly you see as the main differences between those projects.
The app I'll be working on will be a Document Management System where documents will be stored as structure data and made available to users to edit in a WYSIWYG fashion. It very close to what the Texture project is doing.
My current thought about each of them:
Substance: the most powerful of them. It doesn't assume you are importing from one format and exporting to another, I mean, using converters you can use whatever you like: JSON, XML, etc. More complicated to get started. Docs are not great, it could be a lot better. It doesn't seem "production-ready" in the sense of a public open source project, maybe because it doesn't reach v1.0? Maybe that's one of the reasons why it's the less known of all, which is a shame considering how good the code seems to be. It has building blocks like sidebar, toolbar, menus so you can create the UI the way you want it without too much code.
quill: most popular of all. Good docs. It seems very well thought and simple. The thing I don't like is that their model is based on operations (Deltas) and not on state. 'parchment' creates a state representation of your model, but it seems to be a in-memory representation and (at least) not meant to import and export from/to it.  It's very easy to get started and have something working, tho.
ProseMirror: it seems like a very capable framework, but looks less flexible (i may be wrong in this one). It has built-in change tracking, something I would need.  Its schema validates you document which is a nice feature to have (allowing/denying structures based on an RegEx-like representation). Good docs.
slate: popular, flexible, easy to setup and customize but too coupled to react. It has it's own model representation based on JSON and it seems you can't use other formats. You'd need to export/import to/from their JSON format if you would like to use another format.

Looking closer, it seems that substance has a versions and changes tracking system. Still hard to figure it out how all the pieces come together. Are you guys planning on writing docs for the 1.0 release? I couldn't find any mention to it in the roadmap nor issues.

+1

I would like to add Draft.js from Facebook to that comparison list. I will try to migrate from it to Substance. Draft.js is limited to React as it uses React and its built-in functionality. It uses its own JSON format so I will have to figure out some migration script. Another downside of Draft.js is that you need to build all of the UI and most of the functionality yourself (though you have packages to help with that), so it takes a lot of effort to get something basic up and running, but on the other hand, you can customize it pretty well.

The big issue with Draft.js is that it doesn't support mobile browsers, and althrough their documentation states that they are working on it, they have stated that they (facebook employees) are not working on it and won't be. Search for "android" or "ios" in their issue tracker and you'll see what I mean.
On the other hand, substance is not very mature either. It currently only works well on Chrome desktop, but at least they haven't said it is not a goal of this project to support mobiles....

@guilhermeaiolfi - what did you decide to use? I too am evaluating WYSIWYG editors for an app that I am building.

I haven't reached the part where something like substance would be handy.
But for the tests I've made so far, substance would be my choice.
I'm interested in other conclusions as well.
Em 9 de fev de 2018 12:29, "Mark L"<EMAIL_ADDRESS>escreveu:
@guilhermeaiolfi https://github.com/guilhermeaiolfi - what did you decide
to use? I too am evaluating WYSIWYG editors for an app that I am building.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/substance/issues/1127#issuecomment-364448843,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPhmO8_VMVTH8TP2E2Pp8pRrXthXasjks5tTFZlgaJpZM4PykoO
.

I have a few needs - I'm not yet sure how Substance supports them.

I need a text editor that I can embed into an EmberJS app, so embedding needs to be supported. The app is not just an editor, so the editor is not the thing.
I need to import and export text into the editor.
I need to save/load the text into a database.
I am not using React, so I'd rather not include that kitchen sink in my application.
I need it to be extensible so that I can create my own custom features. Specifically, I need to select some text and then create a new object based on that text. Similar I guess to the footnotes and citations examples in Texture project.
I need support for '#hashtags'.
I need support for '@mentions'.
I need support for embedded tables.
I need an open source solution that is actively maintained in 2018.

First, notice that I appear as a contributor because I made a pull request that got merged, but I am not affiliated with this project in any way.
Now I'll try to cover your points:

I need a text editor that I can embed into an EmberJS app, so embedding needs to be supported. The app is not just an editor, so the editor is not the thing.

That's how I'm using substance. I built a customized version of the editor as a separate project, and then use that as an npm module in my app.

I need to import and export text into the editor.
I need to save/load the text into a database.

These are closely related. I'm doing it without much trouble, with a benefit and a drawback. I'm sending just the editor's JSON representation of the document to my backend service. That service renders the HTML itself and caches it (validating the input in the process, and making XSS injections impossible without requiring any complicated HTML purification/whitelisting libraries). It works very well, but there's the added work of coding the renderer server-side.

I am not using React, so I'd rather not include that kitchen sink in my application.

Substance doesn't use react. Nonetheless, React weighs in at 100Kbs minfied but not gzipped. By comparison, my customized version of substance weights 550Kbs minified but not gzipped. I haven't made any serious effort at trimming that size yet, but I don't expect an order of magnitude reduction. What I'm trying to say here is that a proper editor is going to be heavy, and whether it uses react or not should most probably not be an important point.

I need it to be extensible so that I can create my own custom features. Specifically, I need to select some text and then create a new object based on that text. Similar I guess to the footnotes and citations examples in Texture project.
I need support for '#hashtags'.
I need support for '@mentions'.

All "editor framework" projects either directly support that, or provide the "hooks" to build the functionality yourself. This includes Substance, but also ProseMirror, Draft.js and probably some others.

I need support for embedded tables.

Same as above, but probably in a "sadder" state. Tables are hard.

I need an open source solution that is actively maintained in 2018.

I can't speak about the others, but Substance is maintaned according to my own experience. I just started tinkering with it last week, found some issues and opened tickets/PRs. All of them were replied and/or handled promptly, so kudos to you devs!


send log messages from several modules

I want to make a logging function that handles messages from multiple modules in my program. Basically, I want to mimic console.log() that I can use anywhere, but with the pleasant formatting options that come from Winston.
I read in the docs that you could do the following for each file/module that wants to output to the log stream:
var winston = require("winston");
var logger = winston.loggers.get(winston.transports.Console);
logger.info('string to be logged');

I tried this: the log messages always appear, but sometimes don't have the custom formatting that I created initially. (But on some messages, the formatting is there - that's probably a related mystery.)
My real question:

What's a straightforward way to allow multiple modules to output to the same Winston log stream, using the same formatting?

Answered my own question... It's way simpler than I thought - the default options cover the simple needs you might have:
I made a file/module named logger.js containing these lines:
var winston = require('winston');
var strftime = require('strftime');

winston.remove(winston.transports.Console);     // remove the default options
winston.add(winston.transports.Console, {       // and substitute these
  timestamp: function () { return strftime('%F %T.%L'); },
  formatter: function (options) { // Return string will be passed to winston.
    return options.timestamp() + ' ' + options.level.toUpperCase() + " " + options.message );
  }
});
// add more Winston options if desired

In my main app.js file, I add:
var winston = require('winston');
var logger = require('./logger');   // to force the initialization code (above) to run once

In every other file that needs logging, simply add:
var winston = require('winston');

... and then create log entries with:
winston.info('This is the message string to be logged');

Yep, I believe the best practice (at least what I do when I use Winston), especially with the new 3.0 version, is to declare some singleton logger instance, and have all the code that wants to use that logger just import it.   Same goes for transports or formatters.


Some unused code removals, refactorings and fixes

Hello Paul.
Thanks for your awesome QR Scanner.
As I'm planning to create a Polymer Element based on your qr scanner, I refactored it a little bit to be easier to integrate into a custom element. In that process I also removed some code duplication, unused code and fixed some small bugs (line numbers refer to the current head of the production branch):

line 33: removed unused variable detectedQRCode  / undefined return value
line 109: removed unused variable canvas. Therefore also removed unused id="qr-canvas" in the html markup
line 149: removed unnecessary line self.qrcodeNavigate = ""
line 193-194: removed unused variables videoRect and videoElementScale
lines 196-214: refactored the resizing inside of the loadeddata event into a resize method. The loadeddata event on the other hand had been refactored into a onDimensionsChanged event. Like this, the method can also be called from outside and is now also triggered on window resize. This fixes a bug, where the video didn't resize on window resize and the visible rect within the cameraOverlay and the rect drawn on the canvas became out of sync.
line 203-211: Refactored the computation of the scaleFactor into getDimensions to avoid code duplication between the resizing of the video and the resizing of the canvas image portion in setupVariables. Also removed the case distinction between the video being smaller then the screen or bigger to always cover the whole screen with the video which is already archived by scaleFactor = 1 / Math.min(heightRatio, widthRatio) in both cases (note that if the video is smaller then the screen in one dimension, then 1 / Math.min(heightRatio, widthRatio) is greater then 1 and thus the video is scaled up which I think was the desired effect?).
line 219: the refactoring of the scaleFactor computation into the getDimensions method
line 223: If we get no cameras by the getCameras method, your code still tries to run a camera but can't switch the camera (line 390) therefore I changed the check to <=1.
line 236-242: removed unused method toggleFacingState
line 255: remove unused parameter e
line 258-268: refactored starting and stopping cameras into methods that can be called from outside. Also replaced line this.checked === true by cameraToggleInput.checked === true to fix a bug where on visibilitychange always the first camera resumed recordning instead of the previously used one.
line 230,325,533-534: removed the shouldLayout flag as it looked like legacy code to me. Correct me if I'm wrong.
line 308: set the initial value of cameras to null to be able to know whether we did not recieve any cameras yet or there are just no cameras.
line 352: removed unused return value
line 384: avoid that the same camera is set again for performance and reliability reasons
line 404: add an additional check before requestAnimationFrame whether our camera is actually still running. While also cancelAnimationFrame gets called in line 388 this did not always reliably stop the onframe events for me.
line 424: remove a wrong comment
line 460-461,471-473: removed instance variables wHeight, wWidth, scaleX, scaleY, scaleFactor as these are just used locally in the setupVariables method
line 464-465,566-567: removed unnecessary variables dx and dy, which are always 0.
line 479: removed unused variable overlayCoords
line 474,482,486,490: removed the coordinatesHaveChanged flag and don't call setupVariables and drawOverlay now on every frame anymore but only on resizes or video dimension changes
line 507: overlayDimensions as parameter instead of width and height to avoid double computation of the dimensions
line 511-527: remove unused variables boxHeightSize and boxHeightWidth and setting the attributes on the unused overlayCoords. Remove the check for the removed coordinatesHaveChanged.
line 530,536-537: rename setupVariables to resize which now can also be called from outside. Replace unused parameter e. Make all size computations relative to the container of the qr scanner to allow it to be potentially smaller then the full window.
line 546-556,559: use refactored scaleFactor computation instead of code duplication
line 569-570: remove unnecessary initialization
line 580: remove unused return value and call drawOverlay which is now only called on calls to resize.
line 583-586: call the resize method on window resize and video dimensions changed
line 88-95 in camera.css: overflow:hidden for videos bigger then the container to avoid that the container grows

I hope that some of those changes are useful for you.
Kind regards and thanks,
Daniel

I will get to this. I promise.

I appreciate it, thank you :)

This is live now... Sorry for the delay.... and a huge huge thank you!


Calculated Formula Column Value Convert decimal to int

Prerequisites

[x ] Have read and understood: https://github.com/Maatwebsite/Laravel-Excel/blob/3.1/.github/SUPPORT.md
[x ] Checked if your Laravel Excel version is still supported: https://docs.laravel-excel.com/3.1/getting-started/support.html#supported-versions
[x ]  Checked that your question isn't already asked before.

Versions
PHP : "^7.1.3",
Laravel : "5.8.*",
Maatwebsite Excel : "^3.1",
Description
Is is possible to  get calculated formula value without converting to integer.
Additional Information
In excel, I have columns named Country, Country Name, Currency, End Rates and Average Rates. End Rates and Average Rates are Calculate Formula Columns. When I try to import data from excel sheet I'm facing some problem. When I get End Rates and Average Rate value, values are converted to integer. I try to use WithoutIntelligentFormatting and \PhpOffice\PhpSpreadsheet\Cell\StringValueBinder, but can't solve. My question is how to  get calculated formula value without converting to integer.
my excel format

Excel data result

this is my code:
class MySheet implements ToCollection, WithHeadingRow,WithCalculatedFormulas
    {
        public function collection(Collection $rows){
            //Do Something
        }

        public function headingRow(): int
        {
            return 6;
        }
    }

class ImportCurrency   implements WithMultipleSheets
{

    public function sheets(): array
    {
        return [
            'Rates' => new MySheet ()
        ];
    }
}
class CurrencyImportController extends Controller
{
 public function importingCurrency(Request $request){
        $data = json_decode($request['data']);
        $file = $request->file;

        if($request->hasFile('file')){
            $filename = 'excel'. '_'.Carbon::now()->timestamp.'.'. $file->getClientOriginalExtension();
            return Excel::toArray(new ImportCurrency ,$request->file('file'));
          }

}

I'm afraid it's PhpSpreadsheet who does that. Try to reproduce it with PhpSpreadsheet and report it there.


[Refactor] reduce sending audit stats in exchange

Why I'm doing:
sending audit stats for each chunk request takes 27% of time in high concurrent workloads.
What I'm doing:
reduce sending audit stats every _sent_audit_stats_frequency.
_sent_audit_stats_frequency = max(64, RoundUpToPowerOfTwo(fragment_ctx->num_drivers() * 4))), it is scale up with drivers' num. It help to know large query' audit stats even missing the last chunk.
the last chunk of exchange should sent audit stats.
results:
reduce 4% cost to null
Fixes https://github.com/StarRocks/starrocks/issues/37082
What type of PR is this:

[x] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[x] 3.1
[x] 3.0
[x] 2.5

@mergify backport branch-2.5

@mergify backport branch-3.0

@mergify backport branch-3.1

@mergify backport branch-3.2


Fix issues 130 and 131

What this PR does / why we need it:

Fix issues 130 and 131

enable containersecuritycontext and resources limit for initcontainers
Pre-submission checklist:

[x] Did you explain what problem does this PR solve? Or what new features have been added?
[x] Have you updated the readme?
[x] Is this PR backward compatible? If it is not backward compatible, please discuss open a ticket first

:( why must the configuration name be changed.
This is a breaking change


68 multiple entries on go to definition

Closes #68
@karlhorky Here is the first draft to help avoid the extra definitions results from declaration files
I m using a small typescript plugin to help this cause.
currently the option to override the definitions from declaration files can be done to settings Which is documented in the readme
Let me know what you think about this PR

Seems like a really cool approach!
I suggested a few changes and asked some questions

Thanks a lot for the comments @karlhorky . Really Appreciate your time and effort.

I tried this out just now in v1.6.2, but I'm still getting the duplicate entries:

More details:

Page: packages/fusion.upleveled.io/app/(auth)/login/page.tsx
CSS Module File: packages/fusion.upleveled.io/app/(auth)/login/page.module.scss
Next.js type definitions: node_modules/next/types/global.d.ts

It seems like in the monorepo example, the packages/next-app workspace is not configured in package.json, wonder if this is related:
https://github.com/Viijay-Kr/react-ts-css/blob/19198f9c574550011a1df3e61ed2df3e04c91cfb/examples/monorepo/package.json#L5-L10

If I should open a new issue, let me know.

@karlhorky I too tried it after the version was shipped . It did not work for me either. I suspect it could be something to do with the typescript version difference .
I did not have time to investigate this further.
Hope I will find some time to debug this.

I'll create an issue so that we don't lose track of it.

It seems like in the monorepo example, the packages/next-app workspace is not configured in package.json, wonder if this is related:
https://github.com/Viijay-Kr/react-ts-css/blob/19198f9c574550011a1df3e61ed2df3e04c91cfb/examples/monorepo/package.json#L5-L10
Eg. could that mean that the node_modules/next/types/global.d.ts is not being hoisted to the root?

This is not related .
I have removed the next app from packages in order for the package to have it own node_modules folder .
If you look at vite example it has its own node modules setup.So this helped to test the use case you mentioned
However removing the next app from the list of packages didn't make a difference since the monorepo was setup using turbo repo which provides a next boilerplate

I reopened #68 and added a comment: https://github.com/Viijay-Kr/react-ts-css/issues/68#issuecomment-1412524350


Feature Request: Easy import from Todoist

I am currently using Todoist but I want to switch and found Vikunja and I love it so far.
I would just love if I would have a possibility to import my tasks from Todoist with ease. I have over 100 tasks so to write down everything would not be the best idea.

This can be done through the import feature in the desktop app or web frontend. Are you selfhosting or are you using vikunja cloud?
If you are self hosting, you can check out this post to see how to configure the migrator: link
Or check out the part you need to modify in the config example: link
The flutter app (the repo you opened this issue on) will probably not support this in the near future.

This can be done through the import feature in the desktop app or web frontend. Are you selfhosting or are you using vikunja cloud? If you are self hosting, you can check out this post to see how to configure the migrator: link Or check out the part you need to modify in the config example: link The flutter app (the repo you opened this issue on) will probably not support this in the near future.

Got it, I didn't see it so I thought that Vikunja only supports migration from Vikunja Export and TickTick.
I am using the self-hosted version within a Docker container.
Thank you for your quick response, I'll close it since you showed me how to import my data :) Have a nice day and happy neww year!


Notion - Not able to create pages with large content (~2700 characters).

Steps to reproduce the problem:

Create an agent with notion bundle.
Asked to Come up with a story about 3 beautiful duckings
Asked to Write this into a "Duckling" page under "my new Test page"

The tool hits content length restriction.

Debug logs - calls - 2024-11-18T173459.831.json
Following error is seen for the tool call:
body failed validation: body.children[0].paragraph.rich_text[0].text.content.length should be ≤ `2000`, instead was `2780`

If i say - "Yes", it creates multiple pages.

Debug logs - calls - 2024-11-18T173541.895.json
I am able to create a notion page with the contents provided by LLM in step2 using Notion UI with out any problems.
Expected Behavior:
Should be able to create a notion page that includes  all the contents from step2.

@sangee2004 https://github.com/otto8-ai/tools/pull/266 please confirm if PR#266 fixed this issue.

@tybalex you should move your issues to testing when theyre completed

Able to create 1 notion page with large content ( ~4000 characters) when testing with latest version of the tool.


Myscale/vectorstore patch

Description
We added meta-data filtering & Hybrid search to MyScale now.
NOTE: Hybrid search for MyScale is currently implemented as an inline two-staged search hybrid with text match.
special thanks to: @suchme19
Type of Change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Added new notebook (that tests end-to-end)

Suggested Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Awesome! Looks good!
I can't make edits to the PR. Do you mind updating the changelog, just adding this under unreleased new features?
- Added metadata filtering and hybrid search to MyScale vector store (#7780)

ah, I can add it in another pr, no worries


subscriptions: Sort the current user to the top of stream subscribers list.

Followup of #8973
Current user should be append to top of stream subscribers list when user got subscribed and stream subscription modal is open. Currently it's working for public streams subscriptions, but not for private streams. See this comment for more info on why.
To reproduce the issue:

Login with Cordelia in one tab and Iago in another
Iago: Create private stream both subscribed to
Cordelia: Open stream setting page of newly created private stream
Iago: Unsubscribe Cordelia from private stream
Iago: Subscribe Cordelia back to newly created private stream
Cordelia: Check subscribers table in created private stream

Cordelia should be on top of subscribers list, as Cordelia is current user. If you subscribe yourself to public stream, you can check that current user will be on top of subscribers list, we want the same behavior for private stream too.
Though on reload, subscribers get sorted. @rishig Does it make sense to pin the current user at top of all stream subscribers always, if subscribed?

yeah, that seems reasonable.


Refactor ICO unit tests to use Test Runner

Summary or solution description
Part of an overall refactoring.
Changed the execution tests of the ICO example to use TestRunner instead of TestEngine
Tests
Rerun the modified unit tests

Task linked: CU-864ebcrrq Substituir a TestEngine nos testes unitários do smart contract de exemplo de ICO

Coverage: 91.016% (+0.02%) from 90.992% when pulling 1d12b45de5b02806c4a2b103e181c7b2dc93b6b4 on CU-864ebcrrq into ab71cd53dd035cab88ce39368bed8557b71a3b88 on development.


[SIP-1] Split the Table visualization into two visualizations

SIP-1 Split the Table visualization into two visualizations
Motivation
The current Table visualization is confusing as it shows 2 control sections, GROUP BY and NOT GROUPED BY that are mutually exclusive. It's not clear that they are, and if both sections are used Superset will raise and error suggesting to clear some of the controls
Proposed Change
Split the current table visualization into 2 entry points: Table - Samples and Table - Aggregated. The GROUP BY and NOT GROUPED BY control sections will become a Query section in respective visualizations.
New or Changed Public Interfaces
The main change to users is the 2 entries points, essentially a new visualization in the visualizations menu.
Migration Plan and Compatibility

a database migration re-assigning NOT GROUPED BY visualizations to the new type
minor code changes introducing the new visualization

Rejected Alternatives
N/A
@betodealmeida what do you think of this template for SIPs? Took it from here: https://cwiki.apache.org/confluence/display/KAFKA/Kafka+Improvement+Proposals
test edit

@vylc what do you think of the template? Also what do you think about the labels: Table - Samples and Table - Aggregated?

lets try out the template.
With re: to naming:
Table - Samples --> Table - Data Preview, Table - Preview, Table Data
Table - Aggregated --> Table - Data Summary, Table - Summary

@williaster @john-bodley @graceguo-supercat thoughts on this one? I'd like to start a PR doing this soon if people think it's the right approach.

@mistercrunch we agree that the current table UX is confusing, however we aren’t sure if this is really the long-term solution. The atomic vs. non-atomic issue is systemic to all visualization types, not just to the table. i.e., if you extrapolate this logic, we would have to have two time series charts, two bar charts, etc., to support both atomic tables and aggregated data sources.
It seems like a better solution would be to extend ad-hoc metrics to understand the difference between atomic vs non-atomic, i.e., a metric could be an aggregation on a non-atomic column or merely the value of an atomic column.

@john-bodley do you guys have stats atomic level usage (vs agg) and on what viz types are most often used in the non-group by case?

@vylc no because the only way to do a non-group by on other vis's at the moment is to do a hacky group by on something like ds to force it to be an atomic level aggregation.

This got tackled!

Didn't know I accidentally fixed SIP No. 1!


Exclude graphql and protobuf files from formatting

Is your feature request related to a problem? Please describe.
The files lib/graphql/graphql_api*.dart and lib/identification/protobuf/*.dart are autogenerated and should not be formatted.
Describe the solution you'd like

Revert the changes to those files done in https://github.com/digitalfabrik/ehrenamtskarte/commit/f44993429b934e2cf9dc67ed829e15069d2eeb57 and https://github.com/digitalfabrik/ehrenamtskarte/commit/434de1458f4d1ba79b89cef5757355df151dff35
Exclude those files from auto formatting in the pre-commit hook

Excluding specific files does not work: https://github.com/dart-lang/dart_style/issues/864
Instead we should gitignore the generated code.

The code is already gitignored.


chore: use terraform 1.5 for acceptance tests

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

Comparison is base (872618d) 59.56% compared to head (9a9e00a) 13.73%.
Report is 16 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #401       +/-   ##
===========================================
- Coverage   59.56%   13.73%   -45.84%
===========================================
  Files          98        8       -90
  Lines       19754      910    -18844
===========================================
- Hits        11767      125    -11642
+ Misses       7684      770     -6914
+ Partials      303       15      -288

see 90 files with indirect coverage changes

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

Was going to split this out from #398 but remembered we can run the job from that PR manually, so splitting this out isn't necessary.


Test runner fails checkout with "directory already exists"

Description

Test runner fails if the CHECKOUT step is "git clone host ." which is the normal way of checking out a repository in ETOS.
I have not analyzed the problem completely yet, so I don't know why.
[2020-11-05 06:18:54] INFO:Getting test directory with identifier 'git clone https://github.com/eiffel-community/etos-api.git . pip install tox wheel pip install -U setuptools'
[2020-11-05 06:18:54] INFO:Directory not found. Created PosixPath('workspace/tmpe_6z5xke')
[2020-11-05 06:18:54] INFO:Change directory to PosixPath('workspace/tmpe_6z5xke')
[2020-11-05 06:18:54] INFO:Checkout script:
eval "$(pyenv init -)"
pyenv shell --unset
git clone https://github.com/eiffel-community/etos-api.git . || exit 1
pip install tox wheel || exit 1
pip install -U setuptools || exit 1
[2020-11-05 06:18:54] DEBUG:Running command: /bin/bash checkout.sh
[2020-11-05 06:18:54] DEBUG:Published 4 messages, 0 have yet to be confirmed, 4 were acked and 0 were nacked
fatal: destination path '.' already exists and is not an empty directory.
[2020-11-05 06:18:55] DEBUG:Return code: 1 (0=Good >0=Bad)
["fatal: destination path '.' already exists and is not an empty directory."]
[2020-11-05 06:18:55] INFO:Returning to PosixPath('workspace')
[2020-11-05 06:18:55] INFO:Returning to PosixPath('/home/etos')
[2020-11-05 06:18:55] INFO:Compress workspace directory
[2020-11-05 06:18:55] DEBUG:changing into '.'
[2020-11-05 06:18:55] INFO:Creating tar archive
[2020-11-05 06:18:55] DEBUG:changing back to '/home/etos'
Motivation

This is the standard way of cloning and the one we recommend in the documentation and must be fixed.
Exemplification

Benefits

Possible Drawbacks

This is due to us moving the checkout.sh to the test directory.
[2020-11-05 06:41:18] DEBUG:Running command: /bin/bash checkout.sh
[2020-11-05 06:41:18] DEBUG:Published 4 messages, 0 have yet to be confirmed, 4 were acked and 0 were nacked
total 12
drwx------ 2 etos etos 4096 Nov  5 06:41 .
drwxr-xr-x 3 etos etos 4096 Nov  5 06:41 ..
-rw-r--r-- 1 etos etos  193 Nov  5 06:41 checkout.sh
/home/etos/workspace/tmp3ncl0kdz
fatal: destination path '.' already exists and is not an empty directory.


Tip or trick to bind youtube-dl to  a specific IP

Hi,
Is there any way to force youtube-dl to use a specific IP in a multiple NIC environment?
Thanks

Starting from youtube-dl 2<IP_ADDRESS>, you can make use of the --source-address option. See our FAQ if you need help updating.

Thanks for the mod!

The change works :)  Running across two connections right now.


Set user lockout after X amount of failed logins (prevent brute force attack)

Set user lockout after X amount of failed logins to protect against bruteforce attacks.

Feature toggle in config
Configurable number of logins in config
Admin can reset user (Is Active true/false)
Email recovery link

🏷 login

This issue would make more sense in https://github.com/dpgaspar/Flask-AppBuilder as that's what's handling our login. You could also achieve this with a custom security manager, overriding the login method.

also, duplicate of https://github.com/apache/incubator-superset/issues/10560

Closing as duplicate.


Add XCM relay

A program which is able to relay XCM message from one connection to another would be useful for debugging or other purposes.
This functionality should either go into the existing 'xcm' command-line tool (which already relays messages from stdin/stdout to an XCM connection), or a new tool 'xcmrelay'.
The tool should be configured with a server socket address (and the associated server and connection socket attributes) and a client address (with the associated connection socket attributes).
The tool should be capable of relaying data over multiple, concurrent, connections.

Resolved by<PHONE_NUMBER>f3297d3327983140f084b1752daaa4.


Output GPS locations

Output GPS locations

User inputs integer x of number of new locations. Program outputs x GPS locations.
Test case 1: check input is integer
Test case 2: check output is GPS location


PennyLane PyQVM support requires Quilc and QVM running with pyQuil>=2.8

For now, to use PennyLane-Forest with the PyQVM, you will need to have the QVM and Quilc servers running. See https://github.com/rigetti/pyquil/issues/924 for more details.

@josh146 The pyQVM doesn't require the QVM to be running, just quilc (for now).

Thanks! have updated the title

This should be fixed in v2.9.1! :)

Closing due to fix implemented in https://github.com/rigetti/pyquil/pull/934


IsarLink to support Null safety

Steps to Reproduce
Please describe exactly how to reproduce the problem you are running into.
My entire app is null safety and i faced this situation using links.
Code sample
//Domain layer
class AEntity{
        final String name;
        final int id;
        final BEntity bEntity;
        AEntity({
              required this.id,
              required this.name,
              required this.bEntity,
        });
 }

class BEntity{
      late String name;
        final int id;
BEntity({required this.id,required this.name});

}

//data layer

@collection
class A {
Id id = Isar.autoincrement;
late String name;
final b = IsarLink<B>();
}

@collection
class B{
Id id = Isar.autoincrement;
late String name;
}

\\Dto

abstract class BDto {
    static BEntity fromIsar(B b){
            return  BEntity(
                       id: a.id,
                       name :  a.name
            );
    }
}

abstract class ADto {
    static AEntity fromIsar(A a){
            return  AEntity(
                       id: a.id,
                       name :  a.name
                       bEntity: BDto.fromIsar(a.b.value) // Cannot assign type class B? to B.
            );
    }
}

/// Establishes a 1:1 relationship with the same or another collection. The
/// target collection is specified by the generic type argument.
abstract class IsarLink<OBJ> implements IsarLinkBase<OBJ> {
  /// Create an empty, unattached link. Make sure to provide the correct
  /// generic argument.
  factory IsarLink() => IsarLinkImpl();

  /// The linked object or `null` if no object is linked.
  OBJ? get value; // I  changed here to work

  /// The linked object or `null` if no object is linked.
  set value(OBJ? obj); // and here but i think is bad
}

Details

Platform: Ubuntu 22.10
Flutter version: [3.7]
Isar version: [3.0.5]

Links can be null so this is not a bug. You just have to handle the case where no object is linked


win-7 pickups error "get_dimensions(): expected a dict, got None".

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
win-7 pickups error "get_dimensions(): expected a dict, got None".

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Rebooting machine to clear the error.


Ap 2353 remove unused content translations

What
Link to story

Ran i18n-tasks unused to identify (potentially) unused translations (this identified a lot of false positives).
Ran bundle exec rake to identify any tests that have been broken by missing translations and replaced the missing translation.
Tested the citizen journey and the provider journey manually for passported and non-passported claims and replaced any missing translations.
I have attempted to check that none of the translations removed are used, but this could do with another pair of eyes and some manual testing.

Checklist
Before you ask people to review this PR:

[ ] Tests and rubocop should be passing: bundle exec rake
[ ] Github should not be reporting conflicts; you should have recently run git rebase master.
[ ] There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
[ ] The PR description should say what you changed and why, with a link to the JIRA story.
[ ] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[ ] You should have checked that the commit messages say why the change was made.

Codecov Report

Merging #2758 (68d00e3) into master (9805daa) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #2758   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          534       534
  Lines        11168     11168
=========================================
  Hits         11168     11168

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 9805daa...68d00e3. Read the comment docs.


Use ImageBitmap in gltf reader when possible

Review + finalize this PR: https://github.com/iTwin/itwinjs-core/pull/3503

#4010

@markschlosseratbentley Make another item to investigate createImageBitmaps on other devices.


PPU - BGT being displayed at wrong point in chart

On the PPU Chart below the item on the far right hand side (GlucoRx) should be located towards the left as it has a lower PPU (not the highest). The labels on the x-axis underneath it seemed to have changed to numbers, which Seb thinks is the issue

I haven't discovered the source of the numeric labels yet, but after discussing with Rich the underlying issue here is dodgy source data. There are two entries for "GlucoRx Nexus (Reagent)_Strips, one with only 52 items but a PPU of 0.57 (the small red dot in the top right) which drags up the mean PPU for that presentation:
 processing_date | pct_id | practice_id | quantity |        ppu
-----------------+--------+-------------+----------+-------------------
 2018-06-01      | 01E    | P81179      |       52 | 0.574038461538461

According to Rich's research, these 52 items are composed of 50 normally price items, plus two items which have been paid out as two packs, giving a massively inflated PPU.
So the issue here seems to be the data rather than the chart, and needs a wider discussion on how to proceed. (Possibly connected to issue #2).

The issue with the numeric labels is just that the chart gets automatically expanded to fit the bubbles which means that the x-axis extends beyond the number of category labels we have supplied for it. So Highcharts fills in with numeric labels. We can work around this by supplying our own list of labels which we pad with empty string values.


Set-DbaLogin - add a param so that check_expiration can be set

Set-DbaLogin - add a param so that check_expiration can be set

Added a new param so that the check_expiration for the login can be set
Added a new integration test

Type of Change

[ ] Bug fix (non-breaking change, fixes # )
[x] New feature (non-breaking change, adds functionality, fixes #7090 )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[x] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[x] Adding code coverage to existing functionality
[x] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[x] Documentation
[ ] Build system

Purpose

Adding the option to set the check_expiration for the login.
Commands to test

See the integration test for the PasswordExpirationEnabled param.

@wsmelton, looks like you are doing a major restructuring of the repo. This PR can wait or if necessary I can resubmit it once the repo is restructured.

@wsmelton, looks like you are doing a major restructuring of the repo. This PR can wait or if necessary I can resubmit it once the repo is restructured.

@wsmelton, friendly reminder on this PR.

@wsmelton, friendly reminder on this PR.

@lancasteradam - my sincerest apology for the delays. this part of the lockdown is positively crushing and I've been running from larger GitHub projects. I believe I've got things a bit more under control now and will work to merge your 🔥 PRs faster. Thanks again for all of the incredible enhancements.

Thank you @potatoqualitee and @jpomfret.


Add simple Cilium Envoy Config CRD example

Small fixes to INSTALLATION.md
Add a Cilium Envoy Config CRD example with instructions

Not sure if example docs should be in the top level directory or with the yamls in the examples directory.
Note that the example script still needs to be tested out :-)

Renamed the examples directory as custom-envoy-listener and moved instructions to there as well (README.md). Also test drove the script in a local VM with Kind.
Marked as ready for review :-)