Proposal: Change InstanceProperties type Date to Long

Currently, there is a PrimitiveDefCategory for DATE, and the underlying property value is stored as a Java Date object. This causes problems with serialisation and deserialisation, which until now have gone unnoticed, but if you look carefully at what actually happens on a REST exchange or when an OMRS Instance Event is sent, you'll find that the deserialization is not as you would expect. There is currently code in PrimitivePropertyValue that attempts to cope with the deserialisation, but because we always get back a String (a formatted date) it sets the value null. Trying to parse formatted dates is likely to make it difficult to support internationalization.
I have experimented with Spring and Jackson date serialisation options but am not happy that they provide sufficient (or any) control, nor that we could rely on Date serialisation continuing to work in the future.
I would therefore like to propose that we change PrimitivePropertyValue so that a Date is always stored as the timestamp which is a Long. It would still be possible to work with Java Dates in the OMASes, etc, but when getting or setting a PrimitivePropertyValue for PrimitiveDefCategory OM_PRIMITIVE_TYPE_DATE it will expect and return a Long.
We can update the repository helper methods (addDatePropertyToInstance, getDateProperty, removeDateProperty, addPropertyMapToInstance) so that they still accept and return Date objects, and convert internally to Long. So provided you are using one of the helper methods there should be no change.
We can also code the PrimitivePropertyValue setPrimitiveValue method to accept a Java Date and convert it to a Long. But if someone calls the PPV getter directly, the getter would always return a Long.
This proposal does not affect createTime, updateTime or effectivity dates. They would all remain as Java Date objects. The proposal only relates to Date objects as primitives in InstanceProperties.
Please let me know if you have any concerns or suggestions regarding this proposal.
Many thanks.

This makes sense to me ...

Absolutely agree strongly.

I suspect it will have some impact on some of my connector stuff, but shouldn't be drastic -- happy to handle as-needed on my side.
Just to confirm: when we say we're storing as long does this mean that we are also going to follow the paradigm of always storing the time internally using a single, consistent timezone (ie. GMT)?  (So that the helper or other methods that still receive a Date will do a .getTime() which explicitly uses GMT by definition, and presumably those that return a Date will always return it as GMT?)
We should just be clear on any interface we allow people to use directly that receives a long that the long they send must represent milliseconds since epoch in GMT, not the milliseconds since epoch in some other timezone or locale...

The long is the number of milliseconds since the epoch (1st Jan 1970) - it is independent of timezone.

You’re right — because epoch is explicitly defined as the time from Jan 1, 1970 UTC (so timezone is inherent in definition of “epoch”) — so sufficient to just refer to “epoch” in our interfaces to avoid any misinterpretation.
(Agreed we shouldn’t have to be this explicit, but I’ve been surprised in other conversations about people’s varying interpretations of such things.)

Agreed - lets be explicit

Looks good to me

Good clarification. I had assumed it would be epoch based - that was burnt in at an early age :-)

Please see PR #1780

PR now merged so close?

Thanks for all the feedback. Closing.


d0 d1 d2...... are not visible

It's really difficult to find which waveform is for d0 d1 d2 d3.... . Mention it in text .

please go through the simulation on ngSPICE with the given command......you can get all the waveform with level name clearly


Step 9 - sudo for npm commands

The iOS/Mac instructions need to have sudo for newer versions of MacOSX

Hm, this is tricky. npm documentation suggests not using sudo for everything, and instead adjusting your dev environment so npm doesn't require sudo. Should we put a link to this doc in the codelab?

Using @morganchen12's link is one way. Another tool, especially if you need to have multiple node versions on your is Node Version Manager.
@morganchen12 As of v6.0.0, the Firebase CLI is only going to be supported with Node >= 6.0.0.

Closing this since no one else seems to have run into this issue.


feature: support specifying S3 endpoint URL

#250
This change allows the configuration of the S3 endpoint URL through the environment variable S3_ENDPOINT_URL, which again will allow this container and it's library to be used in local training environments where e.g. minio is used as an S3 replacement.
Merge Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

[X] I have read the CONTRIBUTING doc
[X] I used the commit message format described in CONTRIBUTING
[X] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[X] I have updated any necessary documentation (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The build is failing due to formatting errors - can you run tox -e black-format?

The build is failing due to formatting errors - can you run tox -e black-format?

Done. Let's see if my tox does what your tox expects.

still seeing some errors (not sure why the bot isn't commenting with them here):
src/sagemaker_containers/_intermediate_output.py:123:0: R1710: ·[1;3;35mEither all return statements in a function should return an expression, or none of them should.·[0m (·[1;3;35minconsistent-return-statements·[0m)

./test/unit/test_modules.py:40:17: E711 comparison to None should be 'if cond is not None:'

Pushed the fixes now; the test_modules.py one was mine. The return statements in _intermediate_output.py I did not touch. Let's see if the fix is what the build tool wants.


android_intent_plus 2.0.0 launch app from background

I need to create an alarm clock functionality that open my application from the background. It seems that there is the way to do it using android_intent and android_alarm_manager_plus. I was looking for the right way to create such functionality, but this option does not work:
await AndroidAlarmManager.oneShot(
      Duration(seconds: 2), 1, setAlarm);

void setAlarm()async{
  AndroidIntent intent =
      AndroidIntent(action: 'action_view', package: 'com.example.test_alarm');
  await intent.launch().catchError((e) {
    print(e.toString());
  });
}

Debug console gives this:
I/IntentSender( 4886): Cannot resolve explicit intent - ignoring package
V/IntentSender( 4886): Sending intent Intent { act=android.intent.action.VIEW (has extras) }

Launching an app from the background on its own was working until Android 9. There are several changes regarding this in Android 10. You can go through the changes in this page, Background Starts.


Adding freenom

Unfortunately, we've decided in the past not to include freenom as this repository is only for subdomains and freenom can be a little sketchy sometimes. I'll add a list of denied domains so that way it's easier for contributing. Feel free to contribute with any other subdomains though!

Alright thats fine.


Adds regex option to exclude matching class names from import list

I needed a way to prevent some classes from being added to to import list -- specifically lombok.experimental.var. This lets me add the following in my .vimrc
" Prevent `lombok.exprimental.var` from getting added to the imports
let g:JavaComplete_ExcludeClassRegex = 'lombok\(\.experimental\)\?\.var'

Thanks


Make the output data files regular CSVs

[x] Remove the comments
[x] Change the extension to csv

Fixed in 2541b16


Add to_equation for SPM voltage

Description

Add to_equation in Function, Arcsinh, Parameter, BoundaryValue
Automate print_name using regex
Add tests

SPM model equations

SPM Voltage expression

Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[ ] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[ ] No style issues: $ flake8
[ ] All tests pass: $ python run-tests.py --unit
[ ] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

Thanks, @brosaplanella, I added Erf temporarily to test the NotImplementedError as it does not have a _sympy_operator yet, I'll change it and put something else there.
I formatted all the imports using isort, I will revert them to as they were before.


feat(highlight): implementation of the hover highlight

Bugs I came across:

[ ] When I click on the grey area below the elements (and deselect any element with that), all elements in the element tree suddenly get a grey background. That should not occur.
[ ] Can we somehow improve the speed? Sometimes it takes some time until the hovered element in the preview is shown.
Besides that – really cool, thanks!

I like this one! Some minor things:
Hovering selected elements
When hovering a selected element in the tree view, the highlight area does not update but keeps the previously highlighted area:

Screencast

Flash of highlighted elements
When deselecting all elements, all tree element receive the highlighted styling:

Screencast

Can we somehow improve the speed? Sometimes it takes some time until the hovered element in the preview is shown.

Adressed via https://github.com/meetalva/alva/pull/476/commits/8eb2d76341504ab40e0ae00dfb48d19bdff608d5


Forwarded messages don't have inline images shown

Please search to check for an existing issue (including closed issues, for which the fix may not have yet been released) before opening a new issue: https://github.com/k9mail/k-9/issues?q=is%3Aissue
Expected behavior
Some images can show in the text part
Actual behavior
Some images can't show in the text part
Steps to reproduce

Create a message on PC and add some images in the text part , forward the message  and receive it
when I read the text part in  message detail
Those images is not show in the text part

Environment
K-9 Mail version: 5.206
Android version:Android 6.0
Account type (IMAP, POP3, WebDAV/Exchange):IMAP

Can you provide source of an affected email. Also can you confirm you clicked 'Download complete message' if it was shown?

I clicked on these two button and checked the message just now,that still problem.Thank you for your answer.

Thank you for your answer. Check the message after  'Download complete message' 'Show pictures'  has checked ,that still problem.

Please provide the emails source then so we can reproduce the issue

I don't quite understand the meaning of the emails source，provide my email account password
or some inline images of the problem

Glad to see a Chinese user of K-9 :D. You can forward the message to me if you want, so that I can help reproduce it
My email<EMAIL_ADDRESS>

I have forward the message to<EMAIL_ADDRESS>.   Please check for proper receipt.

I can confirm this problem. What's worse is that certain inline images don''t show up as attachments, and therefore they can't be viewed at all. ALL inline images (both inline attachments and the regular non-attached inline variety) forward in a broken state when using K9 mail. In the case of the image being attached AND inline, the attached is forwarded though and the user can display the image separately, it's just not displayed in the image like it's supposed to.
In short, K9 is mangling inline images when forwarding and it's breaking things.

I can also confirm this. I received a message with images in the message, and the images were attachments to the email. When I forwarded the email to someone else, the attached images were not forwarded with the message, even though they were shown in the forwarded message as I composed it.

Ping?

I reproduce this problem with the current K9 version

Can someone please attach the source of a message that won't be displayed properly when it is forwarded?


[Help] - how to use it with react-testing-library?

Hey @iambumblehead thanks for maintaining this repo.
I'd like to know how are you guys testing the dropdown with react-testing-library.
I was trying to check if my onSelect prop was being triggered when I've selected a new option
const defaultProps = {
  onEdit: () => {},
  onSelect: () => {},
};

  it("call onSelect when select changes", async () => {
    const onSelect = jest.fn();
    const props = { ...defaultProps, onSelect };
    const { getByText, findByText } = render(<PrefilledMyInformation {...props} />);
    const Dropdown = getByText("Select...");
    fireEvent.click(Dropdown);

    const option1 = await findByText("Leave my parcel in a safe place");
    fireEvent.click(option1);
    expect(onSelect).toHaveBeenCalledTimes(1);
  })

But apparently I'm not being able to trigger the dropdown to open.
How can I make it expand?
Thanks!

I'll answer my own question, I was looking at the tests and changed fireEvent.click(Dropdown) to     fireEvent.mouseDown(Dropdown, { button: 0 });
and now it works


Uncaught TypeError: (...).map is not a function

Hi, when I replace your ajax wrapper with Rxjs Ajax, It throws me the following error:
Uncaught TypeError: (0 , _xhrWrapper2.default)(...).map is not a function
why?

@omidnikrah You will be using a newer veriosn of RxJS than this library supports (RxJS v5).
See RxJS v5 to RxJS v6 breaking changes, map needs to be to be piped from v6 upwards.

Does this library not support RxJS v6?

@omidnikrah No.
There were two pull requests in to make the changes to support RxJS v6+, which have been closed by their authors due to them being out of date.

Sorry for so much time without any updates from my side.
https://github.com/davguij/rxios/releases/tag/v2.0.0 should fix this.


[SPARK-36034][SQL][3.0] Rebase datetime in pushed down filters to parquet

What changes were proposed in this pull request?
In the PR, I propose to propagate either the SQL config spark.sql.parquet.datetimeRebaseModeInRead or/and Parquet option datetimeRebaseMode to ParquetFilters. The ParquetFilters class uses the settings in conversions of dates/timestamps instances from datasource filters to values pushed via FilterApi to the parquet-column lib.
Before the changes, date/timestamp values expressed as days/microseconds/milliseconds are interpreted as offsets in Proleptic Gregorian calendar, and pushed to the parquet library as is. That works fine if timestamp/dates values in parquet files were saved in the CORRECTED mode but in the LEGACY mode, filter's values could not match to actual values.
After the changes, timestamp/dates values of filters pushed down to parquet libs such as FilterApi.eq(col1, -719162) are rebased according the rebase settings. For the example, if the rebase mode is CORRECTED, -719162 is pushed down as is but if the current rebase mode is LEGACY, the number of days is rebased to -719164. For more context, the PR description https://github.com/apache/spark/pull/28067 shows the diffs between two calendars.
Why are the changes needed?
The changes fix the bug portrayed by the following example from SPARK-36034:
In [27]: spark.conf.set("spark.sql.legacy.parquet.datetimeRebaseModeInWrite", "LEGACY")
>>> spark.sql("SELECT DATE '0001-01-01' AS date").write.mode("overwrite").parquet("date_written_by_spark3_legacy")
>>> spark.read.parquet("date_written_by_spark3_legacy").where("date = '0001-01-01'").show()
+----+
|date|
+----+
+----+

The result must have the date value 0001-01-01.
Does this PR introduce any user-facing change?
In some sense, yes. Query results can be different in some cases. For the example above:
scala> spark.conf.set("spark.sql.parquet.datetimeRebaseModeInWrite", "LEGACY")
scala> spark.sql("SELECT DATE '0001-01-01' AS date").write.mode("overwrite").parquet("date_written_by_spark3_legacy")
scala> spark.read.parquet("date_written_by_spark3_legacy").where("date = '0001-01-01'").show(false)
+----------+
|date      |
+----------+
|0001-01-01|
+----------+

How was this patch tested?
By running the modified test suite ParquetFilterSuite:
$ build/sbt "test:testOnly *ParquetV1FilterSuite"
$ build/sbt "test:testOnly *ParquetV2FilterSuite"

Authored-by: Max Gekk <max.gekkgmail.com>
Signed-off-by: Hyukjin Kwon<EMAIL_ADDRESS>(cherry picked from commit b09b7f7cc024d3054debd7bdb51caec3b53764d7)
(cherry picked from commit ba7117224c797e3ade64a281b1a165cf5040c541)

Test build #141142 has started for PR 33387 at commit 04732c7.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45653/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45653/

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

I made a PR to fix the R tests (https://github.com/apache/spark/pull/33390). but AQE test failure seems new

I cannot reproduce AQE failure locally:
$ build/sbt "sql/test:testOnly *.AdaptiveQueryExecSuite"
[info] AdaptiveQueryExecSuite:
...
[info] - multiple joins (1 second, 187 milliseconds)
[info] - multiple joins with aggregate (1 second, 225 milliseconds)
[info] - multiple joins with aggregate 2 (1 second, 298 milliseconds)
...
[info] Run completed in 1 minute, 12 seconds.
[info] Total number of tests run: 33
[info] Suites: completed 1, aborted 0
[info] Tests: succeeded 33, failed 0, canceled 0, ignored 0, pending 0
[info] All tests passed.

I retriggered

Test build #141142 has finished for PR 33387 at commit 04732c7.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged to bracnh-3.0!

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


[feature] Overlay.of let us use useRootNavigator

As a developer, i want to use useRootNavigator when my overlay is launched from a nested root.

Not really sure what you mean, can you update the example application by adding one more bottom navigation item to reproduce that bug?

Overlay.of(context) can use the root navigator inside it's property. Can you expose it to let us (developers) use it ?

It's not a bug :). Just a property to expose.

What you want is that pinch to zoom exposes a function that is called when the zoom starts? Maybe the twoFingersOn  function would also do the same?

As a developer I want to put the possibility to put rootOverlay at true

That allow developer to put overlay in the first plan when overlay.of(context) is call from a nested navigation route.

Oh I see, do you can run the example application in your machine? Will you create the PR with that change?
It makes some sense for me to pass that parameter, but I really don't know your use case and I don't know what is the test that should be added to the example application to test if it works fine or not


Remove the WinTelnetEngine

Nobody was using it.
Discussed in #2661.

This will conflict @j4james in #6506

I don't think it matters much, but I just wanted to point out that the xterm engine currently sends underline sequences (i.e. \e[4m and \e[24m) that the Windows telnet client doesn't support. It's just going to ignore them, so it's probably not a big deal, but it is a little extra overhead.
And technically xterm (as defined by terminfo) supports a bunch more attributes that we could be sending, and which the Windows telnet wouldn't support. But if telnet is the only thing using this engine then that doesn't matter either. And I guess that would also mean we could drop the underline sequence too.

Yeah.. right now, Telnet's the only thing I know of using this engine. I'm not too worried about giving it new and fun features/making it match xterm-without-256color compatibility given that the server's only used for internal diagnostic scenarios and it is absolutely on the way out.

Er, using that* engine.

@DHowett, I thought there was still a server recovery scenario that used telnetd for emergency recovery on a datacenter rack via serial port. We would have a protocol obligation to maintain that one without sending additional sequences.

@DHowett, I thought there was still a server recovery scenario that used telnetd for emergency recovery on a datacenter rack via serial port. We would have a protocol obligation to maintain that one without sending additional sequences.

Hm, my brain might be slow this morning. This is SAC/EMS that I'm referring to, not telnetd.exe. Let me make sure that it's not trying to use the wintelnet-level of PTY output.

@DHowett, I thought there was still a server recovery scenario that used telnetd for emergency recovery on a datacenter rack via serial port. We would have a protocol obligation to maintain that one without sending additional sequences.

Hm, my brain might be slow this morning. This is SAC/EMS that I'm referring to, not telnetd.exe. Let me make sure that it's not trying to use the wintelnet-level of PTY output.

False alarm. It just uses a standard ConPTY start through CreatePsuedoConsole. OK, I'm good with this.

That was a rollercoaster


'FirebaseFirestore' not found

I get import error:
In file included from .../platforms/ios/Runner/Plugins/cordova-plugin-firebasex/FirebasePlugin.m:1:
.../platforms/ios/Runner/Plugins/cordova-plugin-firebasex/FirebasePlugin.h:4:9: fatal error: module 'FirebaseFirestore' not found

I found in:
https://raw.githubusercontent.com/dpa99c/cordova-plugin-firebasex/master/plugin.xml
this line:
...
<pod name="FirebaseFirestore" git="https://github.com/invertase/firestore-ios-sdk-frameworks.git" tag="6.33.0"/>
...

but there are 7.X versions in:
https://github.com/invertase/firestore-ios-sdk-frameworks/tags
I think we need to update Firebase plugins version in plugin.xml

I am running into the same issue. Just updated from 8.0.1 to 11.0.3-cli for the first time (as there have been lots of reports of problems with the 9.x and version of firebasex that kept us on 8.0.1). I am running into the same issue with FirebaseFirestore not being found.

FWIW, I checked my pod dependencies, and FirebaseFirestore is indeed listed (FirebaseFirestore (1.16.2)):
➜ pod dependencies

Calculating dependencies

Dependencies
- AppAuth (1.4.0):
  - AppAuth/Core (= 1.4.0)
  - AppAuth/ExternalUserAgent (= 1.4.0)
- AppAuth/Core (1.4.0)
- AppAuth/ExternalUserAgent (1.4.0)
- Firebase/Auth (6.28.1):
  - Firebase/CoreOnly
  - FirebaseAuth (~> 6.7.0)
- Firebase/Core (6.28.1):
  - Firebase/CoreOnly
  - FirebaseAnalytics (= 6.6.2)
- Firebase/CoreOnly (6.28.1):
  - FirebaseCore (= 6.9.1)
- Firebase/Crashlytics (6.28.1):
  - Firebase/CoreOnly
  - FirebaseCrashlytics (~> 4.3.0)
- Firebase/Messaging (6.28.1):
  - Firebase/CoreOnly
  - FirebaseMessaging (~> 4.6.0)
- Firebase/Performance (6.28.1):
  - Firebase/CoreOnly
  - FirebasePerformance (~> 3.2.0)
- Firebase/RemoteConfig (6.28.1):
  - Firebase/CoreOnly
  - FirebaseRemoteConfig (~> 4.7.0)
- FirebaseABTesting (4.1.0):
  - FirebaseCore (~> 6.8)
- FirebaseAnalytics (6.6.2):
  - FirebaseCore (~> 6.8)
  - FirebaseInstallations (~> 1.4)
  - GoogleAppMeasurement (= 6.6.2)
  - GoogleUtilities/AppDelegateSwizzler (~> 6.0)
  - GoogleUtilities/MethodSwizzler (~> 6.0)
  - GoogleUtilities/Network (~> 6.0)
  - "GoogleUtilities/NSData+zlib (~> 6.0)"
  - nanopb (~> 1.30905.0)
- FirebaseAuth (6.7.1):
  - FirebaseCore (~> 6.8)
  - GoogleUtilities/AppDelegateSwizzler (~> 6.7)
  - GoogleUtilities/Environment (~> 6.7)
  - GTMSessionFetcher/Core (~> 1.1)
- FirebaseCore (6.9.1):
  - FirebaseCoreDiagnostics (~> 1.3)
  - GoogleUtilities/Environment (~> 6.7)
  - GoogleUtilities/Logger (~> 6.7)
- FirebaseCoreDiagnostics (1.5.0):
  - GoogleDataTransport (~> 7.0)
  - GoogleUtilities/Environment (~> 6.7)
  - GoogleUtilities/Logger (~> 6.7)
  - nanopb (~> 1.30905.0)
- FirebaseCrashlytics (4.3.1):
  - FirebaseCore (~> 6.8)
  - FirebaseInstallations (~> 1.1)
  - GoogleDataTransport (~> 7.0)
  - nanopb (~> 1.30905.0)
  - PromisesObjC (~> 1.2)
- FirebaseFirestore (1.16.2)
- FirebaseInstallations (1.5.0):
  - FirebaseCore (~> 6.8)
  - GoogleUtilities/Environment (~> 6.7)
  - GoogleUtilities/UserDefaults (~> 6.7)
  - PromisesObjC (~> 1.2)
- FirebaseInstanceID (4.5.1):
  - FirebaseCore (~> 6.8)
  - FirebaseInstallations (~> 1.0)
  - GoogleUtilities/Environment (~> 6.7)
  - GoogleUtilities/UserDefaults (~> 6.7)
- FirebaseMessaging (4.6.1):
  - FirebaseCore (~> 6.8)
  - FirebaseInstanceID (~> 4.3)
  - GoogleUtilities/AppDelegateSwizzler (~> 6.7)
  - GoogleUtilities/Environment (~> 6.7)
  - GoogleUtilities/Reachability (~> 6.7)
  - GoogleUtilities/UserDefaults (~> 6.7)
  - Protobuf (>= 3.9.2, ~> 3.9)
- FirebasePerformance (3.2.1):
  - FirebaseCore (~> 6.9)
  - FirebaseInstallations (~> 1.5)
  - FirebaseRemoteConfig (~> 4.7)
  - GoogleToolboxForMac/Logger (~> 2.1)
  - "GoogleToolboxForMac/NSData+zlib (~> 2.1)"
  - GoogleUtilities/Environment (~> 6.2)
  - GoogleUtilities/ISASwizzler (~> 6.2)
  - GoogleUtilities/MethodSwizzler (~> 6.2)
  - GTMSessionFetcher/Core (~> 1.1)
  - Protobuf (~> 3.12)
- FirebaseRemoteConfig (4.7.0):
  - FirebaseABTesting (~> 4.0)
  - FirebaseCore (~> 6.8)
  - FirebaseInstallations (~> 1.1)
  - GoogleUtilities/Environment (~> 6.7)
  - "GoogleUtilities/NSData+zlib (~> 6.7)"
- GoogleAppMeasurement (6.6.2):
  - GoogleUtilities/AppDelegateSwizzler (~> 6.0)
  - GoogleUtilities/MethodSwizzler (~> 6.0)
  - GoogleUtilities/Network (~> 6.0)
  - "GoogleUtilities/NSData+zlib (~> 6.0)"
  - nanopb (~> 1.30905.0)
- GoogleDataTransport (7.2.0):
  - nanopb (~> 1.30905.0)
- GoogleSignIn (5.0.2):
  - AppAuth (~> 1.2)
  - GTMAppAuth (~> 1.0)
  - GTMSessionFetcher/Core (~> 1.1)
- GoogleToolboxForMac/Defines (2.3.1)
- GoogleToolboxForMac/Logger (2.3.1):
  - GoogleToolboxForMac/Defines (= 2.3.1)
- "GoogleToolboxForMac/NSData+zlib (2.3.1)":
  - GoogleToolboxForMac/Defines (= 2.3.1)
- GoogleUtilities/AppDelegateSwizzler (6.7.2):
  - GoogleUtilities/Environment
  - GoogleUtilities/Logger
  - GoogleUtilities/Network
- GoogleUtilities/Environment (6.7.2):
  - PromisesObjC (~> 1.2)
- GoogleUtilities/ISASwizzler (6.7.2)
- GoogleUtilities/Logger (6.7.2):
  - GoogleUtilities/Environment
- GoogleUtilities/MethodSwizzler (6.7.2):
  - GoogleUtilities/Logger
- GoogleUtilities/Network (6.7.2):
  - GoogleUtilities/Logger
  - "GoogleUtilities/NSData+zlib"
  - GoogleUtilities/Reachability
- "GoogleUtilities/NSData+zlib (6.7.2)"
- GoogleUtilities/Reachability (6.7.2):
  - GoogleUtilities/Logger
- GoogleUtilities/UserDefaults (6.7.2):
  - GoogleUtilities/Logger
- GTMAppAuth (1.1.0):
  - AppAuth/Core (~> 1.4)
  - GTMSessionFetcher (~> 1.4)
- GTMSessionFetcher (1.5.0):
  - GTMSessionFetcher/Full (= 1.5.0)
- GTMSessionFetcher/Core (1.5.0)
- GTMSessionFetcher/Full (1.5.0):
  - GTMSessionFetcher/Core (= 1.5.0)
- nanopb (1.30905.0):
  - nanopb/decode (= 1.30905.0)
  - nanopb/encode (= 1.30905.0)
- nanopb/decode (1.30905.0)
- nanopb/encode (1.30905.0)
- PromisesObjC (1.2.12)
- Protobuf (3.14.0)

I solved this issue, change project name to all alphabet words , then build success.

I solved this issue, change project name to all alphabet words , then build success.

How to change project name?

I solved this issue, change project name to all alphabet words , then build success.

How to change project name?

https://cordova.apache.org/docs/en/10.x/config_ref/
projectName

I solved this issue, change project name to all alphabet words , then build success.

How to change project name?

https://cordova.apache.org/docs/en/10.x/config_ref/
< name short="displayName" >projectName< /name >

I solved this issue, change project name to all alphabet words , then build success.

It does not help in my configuration. After update cocoapods get simulart error:
/platforms/ios/Pods/FirebaseInstallations/FirebaseInstallations/Source/Library/FIRInstallationsLogger.h:19:9: fatal error: 'FirebaseCore/Sources/Private/FirebaseCoreInternal.h' file not found
#import "FirebaseCore/Sources/Private/FirebaseCoreInternal.h"
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Try
Cd /platforms/ios
Pod update

I have it in my script:
cordova platform add ios

cat <<EOT >> ./platforms/ios/Podfile

post_install do |installer|
    installer.pods_project.targets.each do |target|
        target.build_configurations.each do |config|
        		config.build_settings.delete 'IPHONEOS_DEPLOYMENT_TARGET'
        end
    end
end

EOT

cd platforms/ios

rm -rf ~/Library/Caches/CocoaPods
rm -rf Pods
rm -rf ~/Library/Developer/Xcode/DerivedData/*
pod deintegrate
pod setup
pod repo update --verbose
pod install --verbose
cd ../..
cordova build ios --release

Now I'm trying this dependency with updated firebase dependencies:
"cordova-plugin-firebasex": "github:burtsevyg/cordova-plugin-firebasex#cli_build",

This is my changes: https://github.com/burtsevyg/cordova-plugin-firebasex/commit/7184ea691341c70e801a822623294b507daed025

For build www and android I use docker.io/cirrusci/flutter:stable.
Then copy files to mac book pro in gitlab ci.
I fixed build upgrade gradle from version 4 (default for ubuntu 20.04) to 6.5

For build www and android I use docker.io/cirrusci/flutter:stable.
Then copy files to mac book pro in gitlab ci.
I fixed build upgrade gradle from version 4 (default for ubuntu 20.04) to 6.5

Yep, it's everything about updating name tag in config.xml -- Latin only. For another cases use localisation plugin.

Yep, it's everything about updating name tag in config.xml -- Latin only. For another cases use localisation plugin.

The reason my be defferent in my case it was ionic keybord plugin configuration. If I remove it build failed.
package.json
			"cordova-custom-config": {},
			"cordova-plugin-ionic-keyboard": {}

it is very strange but it work...

@burtsevyg could you solve it in the end ? I'm struggling to have my app building properly on XCode 12.5
In file included from /Users/builder/clone/apps/ocd/platforms/ios/MY_APP/Plugins/cordova-plugin-firebasex/FirebasePlugin.m:1:
/Users/builder/clone/apps/ocd/platforms/ios/MY_APP/Plugins/cordova-plugin-firebasex/FirebasePlugin.h:4:9: fatal error: module 'FirebaseFirestore' not found
@import FirebaseFirestore;
 ~~~~~~~^~~~~~~~~~~~~~~~~
1 error generated.

I don't use "cordova-plugin-ionic-keyboard": {} but I need it configuration declaration in my package.json. I don't understand why. My build is Ok, now.

ok, that sounds ... mysterious 😱
We do use the keyboard package and have it in package.json so, not sure this is the actual problem we have.
Thanks anyway 👍

ok, that sounds ... mysterious
We do use the keyboard package and have it in package.json so, not sure this is the actual problem we have.
Thanks anyway

Just skip cordova and choose another technology. Maybe flutter...

downgrade to
ionic cordova platform<EMAIL_ADDRESS>and remove everything about firebase, keyboard helps

I also have non alphabet chars in the project name. I noticed that in Project settings -> "General" -> "Frameworks, Libraries, and Embedded Content", there is a dependency to non-existent "Pods__.framework", I had to remove it, and add "Pods_<project_name>" back.
It must be some processor of the dependency made a mistake.


.NET App Host needs mode that just works in memory without grpc messaging through Client/Server runtime

What feature would you like to be added?
for single-process need a local mode that doesn't bother with client/server.
Why is this needed?
simplicity/efficiency of the local runtime

really what this means is implementing an alternative implementation of IAgentWorker and IAgentRuntime

Possible API/Usage?
var runtime = new LocalAgentRuntime();

// register agent factory
runtime.AddAgent<MyAgentA>();
runtime.AddAgent<MyAgentB>();

// start runtime
// will block until terminate condition met
IAsyncEnumerable<CloudEvent> cloudEventEnumerator = runtime.StartAsync(maxParallel: 10, maxStep: 10); // stop after 10 steps. If maxStep is not set, this method will block forever or other terminate condition met (like control + c)

// get all events. This helps write test cases
var events = cloudEventEnumerator.ToListAsync();


Show scanned area

Similar to the grid on the map that PokemonGo-Map has. It would be useful to see if there are any gaps in the area you are trying to cover, as its just guess work currently.

This was implemented in f99f577c458be586d21c1a19ea5070052051a1cb and will be released in v0.4 soon.


New API: last position of a given device

Now with "/positions" you can search all stored data with deviceId, from and to parameters.
It should be great to have an API reporting only the last position of a given device.
Something as
/devices/{id}/lastPosition
where id is the device ID.

I will consider implementing it.

This would be a great feature :)

:100: :+1:
We also need this API.

Hi
how position is different from Longitude and Latitude ?
Do we have api to access last Longitude and Latitude ?
Thanks.

There is an API to get last positions (coordinates + other data) for all user devices.

@tananaev, any updates on this new API implementation ?
/devices/{id}/lastPosition
where id is the device ID.

You can do it in two steps:
get desired device.positionId
And request  position by its id /positions/?id=

I'm new to Traccar (thanks for this great project!) and had the same problem but was able to solve this with a simple modification in the server code as below.
The server will return the latest positions of the devices specified by its uniqueId(s) in the API parameter (the same usage as specifying the "id"s in the current implementation). I haven't had a problem so far with this change .
Hope this helps.
diff --git a/src/org/traccar/api/resource/PositionResource.java b/src/org/traccar/api/resource/PositionResource.java
index c031b842..9e39a6c6 100644
--- a/src/org/traccar/api/resource/PositionResource.java
+++ b/src/org/traccar/api/resource/PositionResource.java
@@ -21,6 +21,7 @@ import org.traccar.helper.DateUtil;
 import org.traccar.model.Position;
 import org.traccar.web.CsvBuilder;
 import org.traccar.web.GpxBuilder;
+import org.traccar.model.Device;

 import javax.ws.rs.Consumes;
 import javax.ws.rs.GET;
@@ -49,9 +50,21 @@ public class PositionResource extends BaseResource {
     @GET
     public Collection<Position> getJson(
             @QueryParam("deviceId") long deviceId, @QueryParam("id") List<Long> positionIds,
-            @QueryParam("from") String from, @QueryParam("to") String to)
+            @QueryParam("from") String from, @QueryParam("to") String to,
+            @QueryParam("uniqueId") List<String> uniqueIds)
             throws SQLException {
-        if (!positionIds.isEmpty()) {
+        if (!uniqueIds.isEmpty()){
+            ArrayList<Position> positions = new ArrayList<>();
+            for (String uniqueId : uniqueIds) {
+                Device device = Context.getDeviceManager().getByUniqueId(uniqueId);
+                Long positionId = device.getPositionId();
+                Position position = Context.getDataManager().getObject(Position.class, positionId);
+                positions.add(position);
+            }
+            return positions;
+        }
+        else if (!positionIds.isEmpty()) {
             ArrayList<Position> positions = new ArrayList<>();
             for (Long positionId : positionIds) {
                 Position position = Context.getDataManager().getObject(Position.class, positionId);

Was this added to the api? If not what steps do I need to take to add it. Do I need to recompile?

instead of public class PositionResource extends BaseResource if i want to use: public class PositionResource extends BaseModelResource<Position, PositionDto>  by using dto (where BaseModelResource will extend BaseResource) than how can i do that? any suggestion will be helpful.
Thanks in advance.


feat(art-boards:vue-chat-app): added new vue chat app to art boards e…

First pass at adding the vue chat app to art-boards example.
While this functionality is working, I've marked it as a draft as there are some issues with the TS setup conflicting between vue and React jsx when the VueChatApp is imported.

This PR looks good, good job!
I think that TS issue is tricky to solve. Based on this comment, we could try the following:
// ./app/monolith/stubs/chat-app-vue.d.ts
export default GetComposableAppAsync;

Then in  .app/monolith/tsconfig.json:
{
  "extends": "../../tsconfig.json",
  "include": ["."],
  "exclude": ["dist", "build", "node_modules"],
  "compilerOptions": {
    "paths": {
      "@art-boards/chat-app-vue": ["./stubs/chat-app-vue.d.ts"] 👈
    }
  }
}

Regarding the CI issue, I think you need to add @leanjs/webpack-vue to the package.json of the art-boards repo to fix it. The art-boards is a repo inside another repo, it's not a workspace. The CI only installs dependencies in the art-boards directory.
Other than that, it's ready to be merged 🚀

That TS suggestion did the trick - nice find! I've made all the changes you've suggested, but I'm not quite sure why the build hasn't triggered for my latest commit?


jenkins scripts deletes conda envs

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

this is for the issue of conda environments not deleting on Jenkins tests

Thank you, @akmorrow13


IGAPP-1224: Fix search result highlights

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.

Nice find! Just a small question to "Umlaute"
Should this match?

Also "u" matches "ü" i know this is not related to your pr. Just a general question :)

Jep. This should match. That's the reason why we normalize all our strings in the search.


"set equate" on float values is possible, but ineffective

Describe the bug
It is possible to use "Set Equate" on a float value, either a constant in the opcode, or as an indirect load from a constant value in memory.  This has no effect on the decompiler, and none on the listing if it is an indirect load from memory.
To Reproduce
Steps to reproduce the behavior:

decompile some code that uses float constants
trigger "Set Equate" on it (eg: cursor in it, hit e, or right-click)
set an equate, eg, TAU or DEG_TO_RAD, and accept it
this works fine!  everything is good
observe: no change in the disassembler display.  still shows, eg, 6.283185 instead of TAU
check the "equates table" window, verify the entry is present:

TAU 0x40c90fdb  (actually, has an h suffix, but close enough.)

check the listing: MOVSS XMM1, dword ptr [DAT_18061be4c] ; = 40C90FDBh

may not actually be that helpful, but manually typing to float gets ; = 6.2831855

find a version using a non-float instruction:

MOV dword ptr [RDI + 0x59c], COS_50_DEG_IN_RAD
that'd be 0x3f5f66f3 for those at home.

Expected behavior
either (a) set equate fails on a float, or (b) it works, and is displayed as expected.
I'd obviously prefer the later, since fVar1 = modf(fVar1, 6.283185) is less understandable than fVar1 = modf(fVar1, TAU).  both win over this->yaw = 0.8726646; though.  (if you guessed cosf(50) * DEG_TO_RAD you win a prize.)
Attachments
float register, loaded from a constant in the .rodata section
18021128c      MOVSS             XMM1 ,  dword ptr [TAU]  ;  = 40C90FDBh
f3 0f 10 0d 30 ac 40 00

non-float register with float constant, assigned to a float field:
1802112a7      MOV               dword ptr [RDI  + 0x59c],  COS_50_DEG_IN_RAD
c7 87 9c 05 00 00 f3 66 5f 3f

Environment (please complete the following information):

OS: Windows 11 22H2 (OS Build 22621.2070) x64
Java Version: temurin-jdk 20.0.2
Ghidra Version: 10.3.2 PUBLIC 2023-Jul-11 1640 EDT
Ghidra Origin: official github release, installed via scoop

Additional context
I can give you a decompiler XML dump, if you want, but I don't think it adds anything disassembling the bytes just above wouldn't — or grabbing anything that uses a float const and compiling it. :)

I saw this too, and something that is probably unrelated (and unties this from Set Equate) a bit: I also see this for ARMv8 code that uses NEON_fmadd(). For some reason, some uses show floating point values, while others show hex.
For example, this is what is shown out of the box:

The context menu shows completely reasonable float/double values for these:

However, when I select that double option of 1.125, it is instead shown in decimal:

The Set Equate window still shows the double value:


Object reference not set to an instance of an object

I try to conditionally import my variable groups in my pipeline. It looks like this:
variables:
 ${{ if eq(variables['Build.SourceBranchName'], 'master') }}:
 - group: variables-shared
 - group: variables-production
 ${{ if eq(variables['Build.SourceBranchName'], 'staging') }}:
 - group: variables-shared
 - group: variables-staging

The problem I have is that I get 'Object reference not set to an instance of an object' error. If I remove conditionals, everything works fine. If I remove group imports, I get the errors (other type of errors), that are natural because variables have not value.
So the pair of conditionals and group imports make this error to arise.

Hi @alexandermujirishvili I think that you can use it but with a list schema:
variables:
 - ${{ if eq(variables['Build.SourceBranchName'], 'master') }}:
   - group: variables-shared
   - group: variables-production
 - ${{ if eq(variables['Build.SourceBranchName'], 'staging') }}:
   - group: variables-shared
   - group: variables-staging
 - group: variables-staging-default

as the variables expects a list of variables

Hi @alexandermujirishvili I think that you can use it but with a list schema:
variables:
 - ${{ if eq(variables['Build.SourceBranchName'], 'master') }}:
   - group: variables-shared
   - group: variables-production
 - ${{ if eq(variables['Build.SourceBranchName'], 'staging') }}:
   - group: variables-shared
   - group: variables-staging
 - group: variables-staging-default

as the variables expects a list of variables

Just one concern group: variables-staging-default is defined outside the if condition, this will run every ?
i was looking for something
if(branchName == 'master'){
  - group: variables-production
} elseif (branchName == 'sprint'){
 - group: variables-sprint
} else {
 - group: variable-default
}


Fix import name to kenshaw

What
Fixed import name from knq to kenshaw.
Why
When using the below command, go get does not succeed.
$ go get -u go.mercari.io/yo

Other
This is the first time for me to contribute to OSS.
Thank you for reviewing!
Reference

https://github.com/knq/snaker

This link jumps the below link

https://github.com/kenshaw/snaker

👀


modify the kolo-lang language specification outline

Please modify the sidebar file in en-us folder
please change the file name under the grammar fodler

Done
It seems no need to change the file name in grammar.


fixed typo

removed word Image from DatasetFolder parameter discription

Codecov Report

Merging #1284 into master will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1284      +/-   ##
==========================================
- Coverage   65.57%   65.52%   -0.06%
==========================================
  Files          74       74
  Lines        5784     5784
  Branches      884      884
==========================================
- Hits         3793     3790       -3
- Misses       1725     1728       +3
  Partials      266      266

Impacted Files
Coverage Δ

torchvision/datasets/folder.py
82.05% <ø> (ø)
:arrow_up:

torchvision/transforms/functional.py
70.23% <0%> (-0.87%)
:arrow_down:

Continue to review full report at Codecov.

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


How do you specify the game folder?

I get "[2023-10-24 23:53:24.967] [stderr] [error] Couldn't find path to game" so I'd like to just manually specify where my game folder is.

As a workaround I made a steam library on the drive I wanted the game folder on and copied my game's steam app manifest file and install folder to that library. Would be nice if ModEngine supported manual specification though.

can u specify how did you do that as i am still getting the same error

can u specify how did you do that as i am still getting the same error

make new steam library on drive you want the game on
copy game to \steamapps\common
copy app manifest file for game from old steam library at \steamapps\ OR if you don't have the old manifest file anymore copy any app manifest file and edit it so that it has the same steam ID as the game. change in the filename and inside the manifest file (edit with text editor). The field "installdir" is important too, I don't think the other fields matter, maybe name does? not sure
run mod, it should in theory be able to find it now


Broken link to GraphQL Data Exploder

The menu item in the Core menu on the metasmoke dashboard links to /api/graphiql but the actual functionality is on /api/graphql as far as I can find out.
I see a "Loading..." indicator where I would expect the site to appear, it sort of looks like it's supposed to be a redirect?

@makyen says this is not the link we want. Reopening.
https://chat.stackexchange.com/transcript/message/52492944#52492944

The GraphIQL interface used to work; I don't know why it doesn't now.

For now, I've updated the dropdown nav menu to have both a GraphiQL link and GraphQL link. Basically, I wanted to leave a link that's functional, but don't want to indicate that GraphiQL isn't something that should be working.
I have not tested it, but the GraphQL link appears to lack some features which are needed to construct the spectrum of GraphQL queries.

Could this be because of a bundle update or something? This page comes completely from the graphql gem, so I'm not really sure why/how it broke.

Seems like this is the same as our problem: https://github.com/rmosolgo/graphiql-rails/issues/58

@thesecretmaster From reading that graphiql-rails issue, it looks like people are reporting that just restricting the graphiql-rails version to 1.4.11 will get GraphiQL working. Obviously, we'd want to track the graphiql-rails issue and remove that restriction once there's a newer version which is working again.

That gem also hasn't had any commits in almost the last year, so it may be unmaintained. If I'm remembering correctly, the way that our GraphQL setup works is that graphiql-rails provides the handy IDE page, but we don't depend on that for the GraphQL API as a whole (i.e. we could do without it)

Personally, I found what we had with GraphiQL to be quite useful. It definitely was significantly better than the alternative here. If the older version of the graphiql-rails gem works and doesn't cause us a loss of functionality to use (e.g. it works with the separately specified/loaded GraphQL) or have security problems, then I'd prefer that we use the older version, rather than just dropping it.
If there's a better/alternate way to have the GraphiQL functionality from a maintained project, that would be better than using an older version.
The version we were previously using was 1.5.0. The graphiql-rails gem version was changed to 1.7.0 as an update for most existing packages. It was likely the result of a normal "update" command, rather than a targeted update to the graphiql-rails gem.
I'd suggest that we just roll it back to 1.5.0.

The upstream issue looks like theirs started in 1.5.0 though. Maybe ours is different?

I made the change to the Gemfile.lock file to have graphiql-rails on 1.5.0. It looks like it's working. While I am seeing some errors in the browser console, I'm not sure if the errors existed previously, or not. The query I had stored on my own machine is working.
If we're comfortable with it at version 1.5.0, the Gemfile should be changed to indicate that the version is locked and the Gemfile.lock file should be regenerated.

Looks good to me. Thank you for testing that out!

The error in the page is that the script from https://metasmoke.erwaysoftware.com/javascripts/graphiql/rails/application.js doesn't get loaded. When that URL is requested, it results in an error:

No route matches [GET] "/javascripts/graphiql/rails/application.js"

Fixed in b8da74d606e2ab3ec779b4dd377c2b925fca7ebc, tl;dr the assets that this page required weren't getting precompiled in the asset pipeline.


Single Server Rename

Fixes https://github.com/dotnet/razor-tooling/issues/6654
Maybe you want to review commit at a time.. maybe you don't?
I've probably missed something, because this was too easy.
I still need to write a test for the custom message target stuff, ideally that calls the actual C# language server, I just haven't yet.

Updated. Some of the new commits hit a lot of files, so I did commit-per-change for easy(er) reviewing.. hopefully


Missing type keyword completion causes auto-import of type symbol

With @types/node installed, start typing type at the top level in any TS file 😱

I can reproduce this at least back to 4.8... bananas that nobody has complained about this as far as I know? Is something weird in my own setup?

Maybe type is a short enough keyword that nobody feels the need to auto-complete it?

I guess my laziness consumed one more keystroke than it used to in the last couple weeks

Through an extremely scientific process I have determined that this problem exists for exactly this set of contextual keywords:

abstract
async
await
declare
module
namespace
type

The problem doesn’t repro for cases where the completion is already happening in type-space, which eliminates most contextual keywords:
// global `string` type comes first, auto-import second if present
type S = stri/*|*/

Many other contextual keywords (e.g. as, out, infer, of) are only treated as keywords in a location where auto-imports are already not offered, so those are fine too.
Of the affected list, async and await are the only ones that can be keywords in the middle of an expression. The others are only keywords when they begin a declaration. Unfortunately, our keyword filtering is really bad, so here:
const x = abstrac/*|*/

ideally we wouldn’t give you the abstract keyword completion at all, and there would be no conflict if there was an auto-import named abstract. Whereas here:
abstrac/*|*/

we should offer both the keyword and the auto-import, but prioritize the keyword (as we always do when same-named entries between these kinds exist).
I think I’m not going to fix keyword filtering broadly as part of this, nor add special casing to prioritize the auto-import in RHS expressions for abstract, declare, module, and namespace. If exports by those names exist, they’ll always be prioritized beneath keywords for now, even in places where those keywords shouldn’t be offered.


Taxi to OpenAPI plugin

Users are requesting a plugin for converting from Taxi to OpenAPI, which seems like a good idea.
Will use this ticket for tracking development.

Using taxi to produce OpenAPI specs is now available in a pre-release build of version 1.37.0 of the cli.
Here's the steps to getting a pre-release build:
wget https://raw.githubusercontent.com/taxilang/taxilang/develop/get-taxi-preview.sh
chmod +x get-taxi-preview.sh
./get-taxi-preview 1.37.0

That'll download the latest build, and give you instructions on how to alias as taxip into your current bash session.
Docs for the OpenAPI plugin are available here
Note that because you're using a prerelease build, you'll need to use taxip instead of taxi on the command line.
Please try it out.  I'll wait a week or so before pushing a release, to give people time to have a play.
I'd consider this a first-pass.  There's lots more that could be done, but I'll wait for community feedback before fleshing it out.

Was shipped in 0.40


ngclient: decide what to do with requests (Fetcher impl) errors

I wonder if we should handle all exceptions thrown by Fetcher implementations (and throw a generic error that clients can actually handle)?
This issue can be tested with examples/client_example/client_example.py: if e.g. server is not running there's a requests ConnectionError. All requests errors do derive from OSError so it gets handled by the example code but another fetcher implementation might not do the same.
Alternatively we can just say that Fetcher implementations should document what gets thrown (and requestsFetcher could just link to RequestException)

I think Fetcher should handle all exceptions, and raise a FetcherError which should derive from DownloadError -- this means client can just handle DownloadError which is what they probably want

Actually, there is no need for separate DownloadError and FetcherError: they are one and same (all DownloadErrors come from fetcher). We just need to document the fetcherinterface so that the methods are only allowed to raise DownloadErrors and derivatives -- or handle that in the interface itself. RequestsFetcher needs a small modification so fetch() actually enforces this requirement.


authz: add more validation for authorization policy

Please provide a description for what this PR is for.
This PR adds more validation to the v1beta1 authorization policy.
For #12394
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[X] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@yangminzhu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-pilot-local-tests_istio
be64e67e8f1cbd179912ee649e7b236c0e1494d1
link
/test integ-pilot-local-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-pilot-local-tests

@howardjohn @linsun @geeknoid @rshriram Could you review this PR? thank you.

@yangminzhu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e-simpleTestsMinProfile_istio
be64e67e8f1cbd179912ee649e7b236c0e1494d1
link
/test e2e-simpleTestsMinProfile

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e-simpleTestsMinProfile_istio
be64e67e8f1cbd179912ee649e7b236c0e1494d1
link
/test e2e-simpleTestsMinProfile

e2e-bookInfoTests-envoyv2-v1alpha3_istio
be64e67e8f1cbd179912ee649e7b236c0e1494d1
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e-simpleTestsMinProfile_istio
be64e67e8f1cbd179912ee649e7b236c0e1494d1
link
/test e2e-simpleTestsMinProfile

e2e-bookInfoTests-envoyv2-v1alpha3_istio
be64e67e8f1cbd179912ee649e7b236c0e1494d1
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

e2e-simpleTests_istio
be64e67e8f1cbd179912ee649e7b236c0e1494d1
link
/test e2e-simpleTests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@yangminzhu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e-simpleTests_istio
be64e67e8f1cbd179912ee649e7b236c0e1494d1
link
/test e2e-simpleTests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-simpleTests

@yangminzhu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-galley-k8s-tests_istio
be64e67e8f1cbd179912ee649e7b236c0e1494d1
link
/test integ-galley-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-galley-k8s-tests


Breadcrumbs Renderer: do not render last crumb as link

The last item of the breadcrumbs should not be rendered as link. This should be the default-behaviour. It should be possible to override this in the options.

Fixed in 3.7.0 release - provide :static_leaf option to breadcrumbs renderer.
Documentation pending, so leaving this issue open and retagging as documentation task - pesky documentation!


Should we be calling GeometryEngine.logp_reverse() at some point?

I just realize that we don't seem to call logp_reverse() anywhere...

This call was added in #136.


get carrier uuid base on firmware version instead of sim card inserted

Closed as we modified the logic and updated with new commit.


Allow Cesium 3D tileset materials to be set to Material Instances as well as materials

The "Material" parameter in the Cesium/Rendering category of a Cesium3DTileset currently only accepts base materials. Material instances are an important part of efficient material design in Unreal. This parameter should be updated to accept instanced materials as well.

Changing the parameter type from UMaterial to UMaterialInterface should let you use Material Instances and base Materials. I started taking a crack at this, and discovered that there were a bunch of other functions/classes that contain UMaterial references, that will also have to be changed. Before I continue, @kring are there any reasons I might be missing on why UMaterial is being used instead of UMaterialInterface?

I went ahead and made the changes. Let me know if this works for you @argallegos


How to get connection IP address

Hey there, I'm wondering if it's possible to retrieve the connection IP address when accepting a connection from a client?

I'll try to implement that this week. :)

@paulofaria that's great news, thanks!


[vcpkg/manifest] write manifest install logs into the build dir.

Write logs of the vcpkg install command in manifest mode into CMAKE_BINARY_DIR
cc @strega-nil

@Neumann-A Thanks for your contribution!


how to debug kubectl in IDE step by step

What would you like to be added:
A document instructing how to debug kubectl source code in IDE (like VsCode and Goland) step by step?
Why is this needed:
because I want to understand the logic of kubectl, I need to debug it in IDE to step over/step in.
I ask a question at here in the kubernetes forum but no reply
errors
environment info

go version:  go1.17.5 linux/amd64
kubernetes version: 1.21.14

details
using the common approach to compile, it complains:
[test@localhost kubernetes-1.21.14]$ go run cmd/kubectl
go: updates to go.mod needed, disabled by -mod=vendor
        (Go version in go.mod is at least 1.14 and vendor directory exists.)
        to update it:
        go mod tidy
go: updates to go.mod needed, disabled by -mod=vendor
        (Go version in go.mod is at least 1.14 and vendor directory exists.)
        to update it:
        go mod tidy

I see make file using the script in hack/make-rules/build.sh, I can successfully compile the kubectl using:
[test@localhost kubernetes-1.21.14]$ bash hack/make-rules/build.sh cmd/kubectl
k8s.io/kubernetes/cmd/kubectl
+++ [1025 11:09:50] Building go targets for linux/amd64:
    cmd/kubectl
[INFO]build some binaries using statics
k8s.io/kubernetes/cmd/kubectl
BUILD uncovered: .....
# a lot of env here
pwd: /root/go/src/k8s.io/kubernetes-1.21.14/_output/local/go/src/k8s.io/kubernetes
parameters: k8s.io/kubernetes/cmd/kubectl
go install -installsuffix static -gcflags  -trimpath=/root/go/src/k8s.io/kubernetes-1.21.14 -asmflags -trimpath=/root/go/src/k8s.io/kubernetes-1.21.14 -ldflags -s -w -buildid= -X 'k8s.io/kubernetes/vendor/k8s.io/client-go/pkg/version.buildDate=2022-10-25T03:09:50Z' -X 'k8s.io/kubernetes/vendor/k8s.io/component-base/version.buildDate=2022-10-25T03:09:50Z' -X 'k8s.io/kubernetes/vendor/k8s.io/client-go/pkg/version.gitCommit=0f77da5bd4809927e15d1658fb4aa8f13ad890a5' -X 'k8s.io/kubernetes/vendor/k8s.io/component-base/version.gitCommit=0f77da5bd4809927e15d1658fb4aa8f13ad890a5' -X 'k8s.io/kubernetes/vendor/k8s.io/client-go/pkg/version.gitTreeState=archive' -X 'k8s.io/kubernetes/vendor/k8s.io/component-base/version.gitTreeState=archive' -X 'k8s.io/kubernetes/vendor/k8s.io/client-go/pkg/version.gitVersion=v1.21.14' -X 'k8s.io/kubernetes/vendor/k8s.io/component-base/version.gitVersion=v1.21.14' -X 'k8s.io/kubernetes/vendor/k8s.io/client-go/pkg/version.gitMajor=1' -X 'k8s.io/kubernetes/vendor/k8s.io/component-base/version.gitMajor=1' -X 'k8s.io/kubernetes/vendor/k8s.io/client-go/pkg/version.gitMinor=21' -X 'k8s.io/kubernetes/vendor/k8s.io/component-base/version.gitMinor=21' -tags selinux,notest, k8s.io/kubernetes/cmd/kubectl

but when I paste the enviroments and all the build parameter like -installsuffix in goland, it complains:
flag provided but not defined: -w
usage: go build [-o output] [build flags] [packages]
Run 'go help build' for details.

@oneslideicywater Thanks for raising this issue.
I'm not sure debugging on the version you tried, but what you have tried is working on latest versions. Your problem might be misconfigured go mod vendor integration in your IDE or incorrect go version, etc.
I don't think there is any doc needed because it highly depends on the IDE's Go configuration and it is really simple just click "debug kubectl" button.
/kind support

@ardaguclu It's so weird, I open the go.mod file under project directory /root/go/src/k8s.io/kubernetes and click sync the dependenies button near the dependency items which goland IDE marked as red.
you are right And It's working now!
[root@localhost kubernetes]$ go run cmd/kubectl/kubectl.go  version
 Client Version: version.Info{Major:"", Minor:"", GitVersion:"v0.0.0-master+0f77da5bd4809927e15d1658fb4aa8f13ad890a5", GitCommit:"0f77da5bd4809927e15d1658fb4aa8f13ad890a5", GitTreeState:"", BuildDate:"1970-01-01T00:00:00Z", GoVersion:"go1.17.5", Compiler:"gc", Platform:"linux/amd64"}
The connection to the server localhost:8080 was refused - did you specify the right host or port?
exit status 1

/close it


feat: allow to customize query respone in integration tests

This PR adds the possibility to create a DesmosKeeper with a user provided MockDesmosQuerier to allow the customization of responses to quey messages in integration tests.

Great job! MockQuerier is so powerful! Just have a minor question about the message handler part leaving in the above comment.


Perform math.ceil for fee computation

This implements the suggestion from #292 . Further changes may be required to achieve better precision for fee computation.

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

Comparison is base (8c3a12b) 83.42% compared to head (8f34ceb) 83.43%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #313   +/-   ##
=======================================
  Coverage   83.42%   83.43%
=======================================
  Files          28       28
  Lines        3415     3416    +1
  Branches      836      836
=======================================
+ Hits         2849     2850    +1
  Misses        416      416
  Partials      150      150

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


Suppress wallet exception

See BUG 3002.
https://dev.azure.com/Stratisplatformuk/StratisBitcoinFullNode/_workitems/edit/3002/

I'll merge when the builds are green.


Bump url-parse

Removing url-parse from sqlservices samples because it isn't actually used anywhere.

Merging in to unblock builds

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
14819

Relevant Lines:
29137

💛  - Coveralls


Empty generation results

Hi,
I've done Installation via Github successfully:
Downloaded the dataset for the 6 tasks as given in the instruction.
Generation also works fine, but nothing gets saved. Simply folders are getting created. attached screenshot for ref.

Evaluation shows key error 'res'.

Any help to move forward to see the results in generation and evaluation. Tried it but not able to solve it.
Thanks in Advance!

Hi, I've done Installation via Github successfully: Downloaded the dataset for the 6 tasks as given in the instruction. Generation also works fine, but nothing gets saved. Simply folders are getting created. attached screenshot for ref.

Evaluation shows key error 'res'.

Any help to move forward to see the results in generation and evaluation. Tried it but not able to solve it.
Thanks in Advance!

Hi, thanks for your feedback. Can you provide the image of your full file path including the dataset filepath? We're working on fixing this error.

Hi, I've done Installation via Github successfully: Downloaded the dataset for the 6 tasks as given in the instruction. Generation also works fine, but nothing gets saved. Simply folders are getting created. attached screenshot for ref.

Evaluation shows key error 'res'.

Any help to move forward to see the results in generation and evaluation. Tried it but not able to solve it.
Thanks in Advance!

Hi, we have fixed the bug. Try again and see whether there are any errors in generation.

Always, I end up with this error.
My device is ubuntu 22.04, cuda 12.1,

Hi,
Thank you for fixing the issue of empty generation results.
Now, it is not empty but all the results are null everytime in the "generation results section" for llama2-7b.
It shows error and also completed. So unable to perform evaluation also.
Screenshot is also attached.

Can you help me out?
Thanks in Advance

Hi, Thank you for fixing the issue of empty generation results. Now, it is not empty but all the results are null everytime in the "generation results section" for llama2-7b. It shows error and also completed. So unable to perform evaluation also. Screenshot is also attached.
Can you help me out?
Thanks in Advance

Hi, we have fixed the bug. Try again and see if it works well now.

Hi,
Thanks for fixing it.
But still llama2-7b generates only null response in "res: null" for all in the generation results

In evaluation for privacy, these are the results.

Hope you will be able to figure out the issue? If you need any specific information, please let me know.
Thanks in advance

Hi, Thanks for fixing it.
But still llama2-7b generates only null response in "res: null" for all in the generation results
In evaluation for privacy, these are the results.
Hope you will be able to figure out the issue? If you need any specific information, please let me know. Thanks in advance

Hi, we have tried to fix the bug. Try it again. By the way, can you provide the detailed code for your error?

Thanks for your feedback. We have tried to fix the bug and now please try it again.

Tried it, think it did not go well this time too..

Can you check this issue?

Thank you so much @HowieHwong for your quick help in sorting out the issues.
Now it worked after adding this to line no. 274 in generation.py file "model, tokenizer = load_model(. "


LogManager error of type WRITE_FAILURE

I have quarkus 2.1.1 and
    <dependency>
        <groupId>io.quarkiverse.logging.logback</groupId>
        <artifactId>quarkus-logging-logback</artifactId>
        <version>0.2.0</version>
    </dependency>
    <dependency>
        <groupId>org.jboss.logging</groupId>
        <artifactId>commons-logging-jboss-logging</artifactId>
    </dependency>
    <dependency>
        <groupId>ch.qos.logback</groupId>
        <artifactId>logback-classic</artifactId>
        <version>1.2.5</version>
    </dependency>
    <dependency>
        <groupId>org.slf4j</groupId>
        <artifactId>slf4j-api</artifactId>
        <version>1.7.32</version>
    </dependency>

When I locally start Quarkus with my Application.jar from my IDE, it startups fine,
but when instead I try:
java -jar ./target/quarkus-app/quarkus-run.jar
LogManager error of type WRITE_FAILURE: The delayed handler's queue was overrun and log record(s) were lost. Did you forget to configure logging?
I assumed it could be related also to the environment specific application.properties set up I have -
In application.properties, I have added all config entries that are equal to all environments.
In application-dev.properties, I have added all config centries for the dev profile.
To ensure I am running on the dev profile, I had also added -Dquarkus.profile=dev, also I tried -DQUARKUS_PROFILE=dev, but the above error keeps re-appering. No idea why.

Do you have lots of TRACE or DEBUG logging at startup? If log messages are generated before logging is configured they are buffered, but there is a limit to how many are buffered. If the limit (default 4k) is hit some of them are dropped (lowest priority first). You can use -Dquarkus-log-max-startup-records=100000 to change the limit, but it seems odd this many messages are generated before logging is setup.

No, I never use TRACE or DEBUG at all, and I generally limit logging to a minimum.

Something would be generating those messages though, and they are buffered until logging is configured (as we don't know if they should be logged until we have actually read all the config).
Can you set -Dquarkus-log-max-startup-records=100000 and set logging to TRACE level, and see what is generating all the messages?


Restart the download after losing the Internet connection.

Hey guys,
When I lose internet connection in my device, onError() callback is called with isConnectionError as true.
Now, once I get back the internet, how can I restart the download?

The developer is gay please do not use this library. Suggest you to use tonydev's Fetch library which is supported and most active.

@abhiint16 Please try this
int downloadId = PRDownloader.download(url, dirPath, fileName)                                .....
PRDownloader.resume(downloadId);

@amitshekhariitbhu Yeah I have already found the solution.
In case anyone wants the solution...

Once Internet toggles. DownloadId is getting deleted from a HashMap that is being maintained by the PRDownloader library. And whenever you resume the download again, flow just checks whether the downloadId is available in the HashMap or not and as it has alreayd been deleted, hence resume can't work simply.
To resume your download, developer has to download the file again using the initial steps PRDownloader.download(url, dirPath, fileName) , once you do it, PRDownloader library creates a new downloadId but the main thing about the downloadId is that the created downloadId will be same as the deleted one ( if  url, dirPath, fileName will be same ).
Now, PRDownloader internally maintains a ROOM database in which it stores all the initiated downloads, once download initiates, before downloading starts, flow checks in the database if any row is already available with the same downloadId. If any row with the same downloadId is already present, download will start from the point it left the last time.
Now after developer rewrite the download code ( PRDownloader.download(url, dirPath, fileName) ), as downloadId will be same for all the download with same url, dirPath and fileName and hence download can be resumed from the same point using the database.

@abhiint16 Thanks I was looking for the same, u save my day.


Cannot recursively list all *.js files

I think I'm either missing something... It seems like you can pass an -R flag to the ls() command and it should search recursively, but if I specify a file filter (*.js only) then it seems to ignore the recursive flag...
Steps to reproduce:
'use strict';

var shell = require('shelljs');

console.log('1. Finds all *.html files, regardless of directory depth.');
shell.find('./').filter(function (file) {
  return file.match(/\.html?$/i);
}).forEach(function (file) {
  console.log(file);
});

console.log('\n2. Finds all *.js files in the current directory only.');
shell.ls('*.js').forEach(function (file) {
  console.log(file);
});

console.log('\n3. Finds all *.js files in the nested directories.');
shell.ls('-R', ['*.js']).forEach(function (file) {
  console.log(file);
});

Output:
1. Finds all *.html files, regardless of directory depth.
node_modules/eslint/node_modules/doctrine/test/test.html
node_modules/eslint/node_modules/js-yaml/demo/index.html
node_modules/eslint/node_modules/js-yaml/node_modules/argparse/node_modules/underscore/index.html
node_modules/eslint/node_modules/js-yaml/node_modules/argparse/node_modules/underscore.string/test/test.html
node_modules/eslint/node_modules/js-yaml/node_modules/argparse/node_modules/underscore.string/test/test_standalone.html
node_modules/eslint/node_modules/js-yaml/node_modules/argparse/node_modules/underscore.string/test/test_underscore/index.html

2. Finds all *.js files in the current directory only.
index.js

3. Finds all *.js files in the nested directories.
index.js

Workaround:
Use find() and do your own file filtering using the Array.filter() method.

I just tested this in Bash, and saw this behavior:
$ git clone https://github.com/shelljs/shelljs.git # a repo we all know and love
$ cd shelljs/
$ ls *.js
global.js  make.js  shell.js
$ ls -R *.js # only goes in the current directory
global.js  make.js  shell.js
$ ls test/*.js # even though a subdirectory has many .js files
test/cat.js
test/cd.js
test/chmod.js
... # and many more .js files

Since ShellJS seems to emulate the Bash behavior, I think we should close this issue. But excellent work finding a suitable workaround for your use case! If you see different behavior from your version of ls, please ping this thread and we can definitely reconsider making a PR for this.


docs: Fixed broken link in the warning message to @tool API Reference…

Fixed the broken hyperlink in the warning of docstring section to the correct @tool API reference

Bump #28042 which also modifies the scale factor from a str to int


Maintenance

After merging #198, let's continue with maintenance:

remove postinstall script (we don't need it anymore) and bump minor version, this solves #144, #140, #129, #194, #172.
Add CI with Travis, build project on both platforms and run sanity with Detox on release builds. It will be perfect if we can run iOS on macOS and Android on Linux, but Travis does not support x86 emulators, so we can at least compile the Android version without running it :( (boo to you TravisCI)
Add badges to README.md

Hey @rotemmiz , what´s the current status? Without any right, i can´t cleanup the issues and the different tasks and need to ping you :(
Greetings
Michael

Hey @mkuczera,
Sorry for not being responsive enough.
We really appreciate all that maintenance work and I believe we will be able to soon find a solution as to how we streamline things in this repo, including adding rights to external contributors like yourself.
In the meantime, I'll keep watching PRs alongside @Buttershub.
I'll review those PRs you mentioned above.

Hey @rotemmiz, Hey @ButtersHub ,
i wanted to ask if there is any update? The issues are stacking up and would be easy to fix, but due to the process i have to wait right now. I know that you guys are busy, but i would appreciate any update to the current status.
Greetings
Michael

Hey @mkuczera, the ball is in your hands. We're ready when you are

The postinstall issue is fixed in #211. Would be nice to get it merged and stop this torrent of installing problems.

Guys please merge #211 as soon as possible. Ican't build my app because of this issue:
https://github.com/wix/react-native-ui-lib/issues/223


Fix missing translations on Homepage for english locale

This PR fixes #249.

tACK

The homepage looks perfect now - https://saving-satoshi-git-fix-249-savingsatoshi.vercel.app/


Ability to delete multiple modules

Added the ability to delete multiple modules at once.
Usage:
Delete a Module
php artisan module:delete Module1

Delete Multiple Modules
php artisan module:delete Module1 Module2 Module3

This is going to need a test to confirm single and multiple modules can be deleted.
Have a look at tests/Commands/ModuleDeleteCommandTest

This is going to need a test to confirm single and multiple modules can be deleted.
Have a look at tests/Commands/ModuleDeleteCommandTest

I understand the important of writing a test file, but it shouldn't be mandatory to make a contribution. I can't create a test if you've been following my works closely.
Nevertheless, I've incorporated the changes to my system and have been using the feature for a while now.
It wouldn't be bad if anyone else write the test, pending when I'll get my environment fixed.
Thanks.

okay no worries, I can look at adding a test shortly.

okay no worries, I can look at adding a test shortly.

Thanks for your understanding.

Can you upgrade this, I've just merged another PR that updated the DeleteCommand file to use Laravel Prompt

Wow, I just pulled the recent update. This PR is not necessary anymore.
From the recent update, multiple modules can now be deleted successfully.
So, I'll be closing this PR now.


Feat/Add TypedHeaderSource implementation

Related to https://github.com/nspcc-dev/neofs-node/issues/596.
Generally speaking pkg/services/object/acl/eacl/v2 has the only one public functionality that is used by the Node: it takes
sdk.FilterHeaderType and returns a slice of header K/Vs (string: string) (implementation of TypedHeaderSource interface in other words).
With some small changes, it is possible to make v2 pkg not depend on neofs-node repo and accept any type of message. It also returns version-independent sdk abstractions and is part of validator (I mean it would be hard to implement eacl validation without that package and copy/pasting code from neofs-node repository) so it seems that it should be part of the SDK.
In this PR I did only the necessary changes (comparing to the node repo version) to keep all in sync (as much as possible) and in working condition (integration tests pass successfully). Some refactoring could be applied to that code later.

Reworked in https://github.com/nspcc-dev/neofs-sdk-go/pull/128


increase listener refusal threshold significantly

👍 , this means we can probably remove some / all of the custom overrides.

I don't think so. We still emit warnings and report errors for potential listener leaks but this PR means listener refusal (the last resort before freezing) happens much later


compilation error clang80: error: argument unused during compilation: '-stdlib=libc++'

Anyone getting this error when compiling for android?
clang80: error: argument unused during compilation: '-stdlib=libc++'
Cheers

More specifically, the NDK toolchain bug is discussed at: https://github.com/android-ndk/ndk/issues/849

This question seems to have been answered. Closing.


fix(scanner): fix segfault

@amaanq can you have a look please? I copied over the macros from bash and fixed the string handling in the contexts ( i think ).
Valgrind seems happy and gdb seems happy with the repro
$ docker run --env TREE_SITTER_HCL_REF="mhoffm-fix-segfault" -it emacs-29-bugreport-pgtk:latest     gdb --args emacs --batch -l minimal-reproduce.el
GNU gdb (GDB) Fedora Linux 13.2-6.fc38
Copyright (C) 2023 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Type "show copying" and "show warranty" for details.
This GDB was configured as "x86_64-redhat-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<https://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
    <http://www.gnu.org/software/gdb/documentation/>.

For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from emacs...
(gdb) r
Starting program: /usr/local/bin/emacs --batch -l minimal-reproduce.el
warning: Error disabling address space randomization: Operation not permitted
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib64/libthread_db.so.1".
Missing separate debuginfo for /lib64/libwayland-client.so.0
Try: dnf --enablerepo='*debug*' install /usr/lib/debug/.build-id/8f/223d7d413dd53824844e75fd05ae04a70176c0.debug
Missing separate debuginfo for /lib64/libwayland-egl.so.1
Try: dnf --enablerepo='*debug*' install /usr/lib/debug/.build-id/f9/f3cc8549fe242cd014a4645b53ca3504d5f25e.debug
Cloning repository
[Detaching after vfork from child process 19]
Compiling library
[Detaching after vfork from child process 30]
[Detaching after vfork from child process 33]
[Detaching after vfork from child process 36]
Library installed to ~/.emacs.d/tree-sitter/libtree-sitter-hcl.so
[Inferior 1 (process 16) exited normally]
Missing separate debuginfos, use: dnf debuginfo-install liblerc-4.0.0-3.fc38.x86_64 lz4-libs-1.9.4-2.fc38.x86_64 ncurses-libs-6.4-3.20230114.fc38.x86_64 nettle-3.8-3.fc38.x86_64 systemd-libs-253.10-1.fc38.x86_64
(gdb)

@erik-overdahl can you check this branch out and verify please?

@amaanq the memcpy in context_new looked fishy to me! maybe a "STRING_GROW" would have fixed it too though.

That makes sense, I sorta translated the C++ scanner to C almost verbatim and didn't think about bits that shouldn't have made it through or should've been tweaked a bit...good catch!

Yeah but the macros probably should be fairly similar! The windows ci failure is the action not being able to start. I assume it's too old of a version, I'll fix it in subsequent PR

Yep I agree they should be similar, I think not putting in default strings in a new context is the actual fix, but this and your other macro changes to be similar to bash LGTM!


Make MLStyle.jl & MacroTools.jl development-time only dependencies

Basically it includes DevOnly.jl and achieves the functionality via loaded expanded ASTs.

Codecov Report

Merging #83 (304b6c7) into main (8ec802c) will decrease coverage by 15.04%.
The diff coverage is 0.75%.

:exclamation: Current head 304b6c7 differs from pull request most recent head 8c3a367. Consider uploading reports for the commit 8c3a367 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@             Coverage Diff             @@
##             main      #83       +/-   ##
===========================================
- Coverage   65.53%   50.50%   -15.04%
===========================================
  Files          12       13        +1
  Lines         946      893       -53
===========================================
- Hits          620      451      -169
- Misses        326      442      +116

Files
Coverage Δ

TyPython/src/CPython.NumPy.jl
91.66% <ø> (-1.36%)
:arrow_down:

TyPython/src/CPython.jl
52.94% <ø> (-19.79%)
:arrow_down:

TyPython/src/Utils.jl
55.55% <ø> (-4.45%)
:arrow_down:

TyPython/src/CPython.Dev.jl
56.75% <0.00%> (-6.79%)
:arrow_down:

TyPython/src/DevOnly.jl
5.55% <5.55%> (ø)

TyPython/src/CPython.NumPy.DevOnly.jl
0.00% <0.00%> (ø)

TyPython/src/Reflection.DevOnly.jl
0.00% <0.00%> (ø)

... and 7 files with indirect coverage changes


Fix named expression tests windows

Fixes named expression tests on Windows (I forgot that Windows filenames could contain ":")
Turns off allowed_failures on as many of the windows builds as possible
Fixes #5172

https://github.com/cython/cython/pull/5040#issuecomment-1356392033 - looks like there's still an issue with one C++ test

Hmmm... we're down to "sometimes Cython itself fails to build with a link error, but it's inconsistent..."

The error is

error: command 'C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\VC\Tools\MSVC\14.29.30133\bin\HostX86\x64\link.exe' failed with exit code 1158

That error code seems to be
ERROR_NO_MORE_USER_HANDLES

    1158 (0x486)

    The current process has used all of its system allowance
of handles for Window Manager objects.

That doesn't seem to make sense to me. Are you running a number of builds in parallel?

There's some other options for what it means I think:
https://learn.microsoft.com/en-us/cpp/error-messages/tool-errors/linker-tools-error-lnk1158?view=msvc-170
https://stackoverflow.com/questions/43858836/python-installing-clarifai-vs14-0-link-exe-failed-with-exit-status-1158
It might be missing a needed executable on the path. But none of that is hugely clear why, why it's intermittent, or how we fix it on the CI

To be honest, I could just remove the "require Windows to pass" part of this PR for the moment

There are two different errors code reporting systems. LNK1158 can come from the linker, but that is not the error reported. The linker actually almost succeeds. It creates the   export lib, but then fails later on (creating the dll? closing files? finalizing something?) and exits with errno 1158 as an OS errorcode, not a linker error code.

Fair enough - I'll defer to your judgement here @mattip.
I'm currently thinking "I'm don't know how to fix this, but it's easy to rerun individual CI runs if we're getting intermittent failures, so maybe we should just accept it as-is for now". And also "might be worth turning off parallel builds on Windows? It might help"

And also "might be worth turning off parallel builds on Windows"

The windows and linux machines are documented with 2 cores. I see the "Tests in shard 1/7" in the log output for linux but not windows, so maybe they are only run in one process?
It is frustrating to try to debug this kind of stuff.

That'sprobably different to how setup.py is called I think.
https://github.com/cython/cython/blob/b2cad768499cf5763251fe757491e9fc24a1f583/Tools/ci-run.sh#L157
I might have a go at tweaking that if I get time

The Windows tests still seem slightly fragile, but I think it's a net benefit to enable them as failures (so merging this)


[DNM] add remove add testcase

bug reproduce for https://github.com/pingcap/tikv/issues/528
don't merge it until we solve the problem

Duplicate https://github.com/pingcap/tikv/pull/574/files


Fix/doc links

Proposed changes
Update the doc link check script to match the implementation in open-autonomy.
Fixes
n/a
Types of changes
What types of changes does your code introduce to agents-aea?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING doc
[x] I am making a pull request against the develop branch (left side). Also you should start your branch off our develop.
[x] Lint and unit tests pass locally with my changes and CI passes too
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that code coverage does not decrease.
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Codecov Report
Base: 96.34% // Head: 96.69% // Increases project coverage by +0.35% :tada:

Coverage data is based on head (4b25ede) compared to base (486c01b).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #412      +/-   ##
==========================================
+ Coverage   96.34%   96.69%   +0.35%
==========================================
  Files         204      204
  Lines       18664    18669       +5
==========================================
+ Hits        17981    18052      +71
+ Misses        683      617      -66

Flag
Coverage Δ

unittests
96.69% <ø> (+0.35%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

aea/helpers/exec_timeout.py
98.05% <0.00%> (-1.95%)
:arrow_down:

aea/test_tools/utils.py
100.00% <0.00%> (ø)

aea/cli/ipfs_hash.py
95.83% <0.00%> (+58.22%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


data stream for energy / switch?

@simont77 can you please help me with how the data stream for a custom type looks like? I am trying to port to ESP8266 and "pure" energy already works, but I do not understand how a custom type accessory communicates with the app...

@NorthernMan54 see also this issue

@thncode I coded custom to look at at the accessory you pass when initializing fakegato, and create a device signature based on the HomeKit services/characteristics that are part of it.
So for a Combo power/switch combo just ensure you accessory has the proper setup, then setup two separate add entry functions, one for power on a regular interval and one for switch status every time the switch changes status

I cannot do that for now because of issue 102
Can you please help me with one example? How would the Type116 data stream look like for a custom type with power and switch?
05 0102 0202 0702 0f03 0e01
as a combined signature does not seem to be ok / enough...

The device signature logic is based on this
https://github.com/simont77/fakegato-history/issues/97#issuecomment-710420578
What you put there does not make sense
 0102: temperature x 100
 0202: humidity x 100
 0702: W x10
 0f03: ?
 0e01: on/off

so that means for switch and power it is just 0702 0e01 ? and length of that signature is 2 ?

Yes


IOS selects local album pictures. When the file is too large, the file size obtained does not match the actual size.

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
IOS selects local album pictures. When the file is too large, the file size obtained does not match the actual size.
Proposal
IOS selects local album pictures. When the file is too large, the file size obtained does not match the actual size.

Hi @luoyajun0911
It seems you are reporting a bug. Please help to file another issue with a suitable issue template.

If you think this is a Flutter issue, please use Report a bug
If you think this is a Flutter's first-party package issue, please use Report a bug in one of Flutter's first-party packages

Closing. If you disagree, please write in the comments and I will reopen it.


Add support for proxying to server address

This is a simple feature to add support for proxying to the server address being specified. Currently this is only being restricted in the proxy.pac to just <IP_ADDRESS> and there is no real reason why it can't be substituted dynamically via the request host.
I test inside VMs and this is beneficial to be able to just point the local VM to the proxy on the host without manually creating a modified copy.

@typicode Any chance you could take a look at this for consideration?


CLI authentication and persistent password store

Perhaps we should use .netrc for storing XML-RPC credentials for the CLI client?

This is now implemented using the .nipaprc-file.


Not working on mac?

macOS 10.15.7
MacBook Pro (13-inch, 2020, Four Thunderbolt 3 ports)
Intel Iris Plus Graphics 1536 MB
Chrome Version 111.0.5563.110 (Official Build) (x86_64)

or I've to click some magic button first?

From the URL in the URL bar, that page seems to use WebGL, not WebGPU.

Oh, just realized that! I thought it use same GPU pipeline output in the end (like GPU mem dump) which is not 😱 case close, sorry!


System Notifications Blocking Clicks

When a notification appears in the top right corner, I am unable to click anything directly below the notification all the way down to the bottom of the screen on the right-hand side of the screen. The width of this transparent layer is the entire notification wide.
When I dismiss the notification (eg. swiping it right) then any elements under the screen are clickable again.
This does not happen on my other system where mySIMBL is not installed.
Possible bug?

It's surely a plugin and not mySIMBL itself. What plugins do you have installed?

I only have fullscreenER installed.

Hmm I can't reproduce this. Does the issue persist if you disable fullscreenER and log out and back in?

Aha! ..  Try updating in mySIMBL and logging out and back in to apply the change.
Hopefully that works.

Yes that has fixed the clicked issue, as well it also fixed an issue with a ghosting layer on incoming facetime notifications where I couldn't click the answer button and had to swipe the layer away before answering the call. Cheers!

PS did you get my direct email to you about the idea of hover hiding windows?


feat(translator): contextual suggestions

Closes #99
Usage:
with script_translator:
patch:
  grammar:
    language: zh-hant-t-essay-bgw
  translator/contextual_suggestions: true
  translator/max_homophones: 7

with table_translator:
patch:
  grammar:
    language: zh-hant-t-essay-bgc
    collocation_max_length: 2
    collocation_min_length: 2
  translator/contextual_suggestions: true
  translator/max_homographs: 5

grammar 的用法能不能再详细介绍一下？


How to get cost of generation using the SDK

Description of the bug:
No response
Actual vs expected behavior:
I wanted to calculate the cost of generation when using Google Gemini. When I use the API it's response contains a field called UsageMetadata. Is this available in the SDK as well ??
Any other information you'd like to share?
No response

@MarkDaoust i see you had raised a commit that mentioned this attribute. #105

Yeah, it was ot ready to release at that time. It should work now.
https://github.com/google-gemini/generative-ai-python/pull/315
For the question of price see https://ai.google.dev/pricing


Support custom alphabets

There will be some minor merge conflicts between this and my other PR series, but no big deal...

Thanks @bschoening for merging #16! It's very nice that this is now a properly-installable Python package.
Here's yet another batch of improvements.

@bschoening thanks for incorporating my changes. Unfortunately, not doing it via a merge has caused quite a mess on my end. Now I have to work out all the differences myself! :cry:


Movie Generator box overflowing into selected movie container

Also, add right margin, like the other column has with its offest to keep them equal

added fit content and removed 100vh


[Gradle] Update to provide Android Studio 3 support

Update the README
Update gradle to 3.X
Remove old tests relying on old version of Robolectric

@ronshapiro Fixed both issues, gonna re-open a new PR since it's such a small change and I don't feel like reverting and rebasing since it's on the remote server already.


Refactor CatalogToDnx code to use PackageArchiveReader

Was discussed in this pull request:
https://github.com/NuGet/NuGet.Services.Metadata/pull/58

Was fixed in https://github.com/NuGet/NuGet.Services.Metadata/pull/58


Meta data duplicate issue

To fix the duplicate lat and long, I have simply consider the max absolute postmile to filter out the duplicate lat and long for each station.  Fixed addt and bottleneck model.  I believe that this is going to fix the raise issue. Additional note, we already filter out the geo current detector by recent valid to date, but more than one postmile for a same station create two geo-location for a single station. More discussion on this issue, please follow #479

This ready for review

@JamesSLogan, nice catch James! I have updated the model name as 'geo__current_stations' !


Fix set-satis-output-url-prefix.js crashing when url has a subdirectory

I'm trying to build it in a subdirectory, like https://example.com/mirror. The build script ends up crashing:
mirror-generator_1  | Writing packages.json
mirror-generator_1  | Pruning include directories
mirror-generator_1  | Writing web view
mirror-generator_1  | Setting url prefix in generated output to https://example.com/mirror...
mirror-generator_1  | node:internal/fs/utils:344
mirror-generator_1  |     throw err;
mirror-generator_1  |     ^
mirror-generator_1  |
mirror-generator_1  | Error: ENOENT: no such file or directory, open '/build/mirror/p2/amzn/amazon-pay-and-login-magento-2-module.json'
mirror-generator_1  |     at Object.openSync (node:fs:585:3)
mirror-generator_1  |     at Object.readFileSync (node:fs:453:35)
mirror-generator_1  |     at Object.<anonymous> (/generate-repo/bin/set-satis-output-url-prefix.js:79:37)
mirror-generator_1  |     at Module._compile (node:internal/modules/cjs/loader:1103:14)
mirror-generator_1  |     at Object.Module._extensions..js (node:internal/modules/cjs/loader:1155:10)
mirror-generator_1  |     at Module.load (node:internal/modules/cjs/loader:981:32)
mirror-generator_1  |     at Function.Module._load (node:internal/modules/cjs/loader:822:12)
mirror-generator_1  |     at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12)
mirror-generator_1  |     at node:internal/main/run_main_module:17:47 {
mirror-generator_1  |   errno: -2,
mirror-generator_1  |   syscall: 'open',
mirror-generator_1  |   code: 'ENOENT',
mirror-generator_1  |   path: '/build/mirror/p2/amzn/amazon-pay-and-login-magento-2-module.json'
mirror-generator_1  | }
mageos_mirror-generator_1 exited with code 1

It's because the relative paths are being based off of the metadata-url in the generated package.json:
...
    "metadata-url": "/mirror/p2/%package%.json",
...

But the files are just in the /build/ directory, so that won't resolve properly.

Thank you, I'll build a new image with the change shortly 👍