Trucation token with empty spaces before is not tappable.

The trucation token here is "...more" .This is an issue we faced with the trucation token if there is empty space before the truncation token it is not tappable. The issue seems to be with the link range calculation done in line : 943, but can be with any other thing also. Kindly resolve this issue. Thanks in advance.

I have been fixed above issue, please ref https://github.com/TTTAttributedLabel/TTTAttributedLabel/issues/688 and give any feedbacks


Extension doesnt seem to be able to find python

Hello,
I'm having an issue using this extension. When i currently try to run the doctest using the extension button, a terminal pops up with the following command:

undefined -m doctest -v "test.py"

when i manually run

python -m doctest -v "test.py"

this works.
I looked at the source code and found that the issue is that pyDoctestBtn is trying to find python.pythonPath which after looking around online for a while has been deprecated.
I can manually add it back, but this doesnt quite work since I would ideally want the python command to be run using the current conda enviroment i am using.
Apologies if this is something that has already been fixed, but i just wanted to point it out since i'd like to keep using this extension!

Looking at the Python Extensions current workspace settings it appears that they changed the setting ID to 'python.defaultInterpreterPath'. I should be able to go in and fix that, but Paul's temporary workaround should do the trick in the meantime.

For now I used some code from vscode-code-runner to attempt to retrieve the python path from a location other than python.pythonPath, and I added a fallback setting to the extension that you can edit to be your python path if the extension cannot find one. If anyone understands how to extract the appropriate python path automatically now that python.pythonPath is depreciated, I am trying to figure it out.


[TRACKER] Compilation warnings raised on macOS builds

Godot version:
3.2 master (5cc1c7d)
OS/device including version:
macOS
Issue description:
This tracker should list all current warnings raised for macOS builds, specifying when relevant the build options (tools, target).
The initial set of warnings has been copied from @bruvzg's report on #29033.
If you're a contributor with edit access, please modify the OP directly to add more checkboxes for new/other warnings. If you don't have edit access, you can add them in a comment below.
warnings=all
With Xcode 10.2.1 (@bruvzg):

[ ] tools=yes target=debug in scene/resources/texture.h (more details in #24420)

scene/resources/texture.h:91:1: warning: class 'BitMap' was previously declared as a struct [-Wmismatched-tags]
class BitMap;
^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/System/Library/Frameworks/ApplicationServices.framework/Frameworks/QD.framework/Headers/Quickdraw.h:61:8: note:
      previous use is here
struct BitMap {
       ^
scene/resources/texture.h:91:1: note: did you mean struct here?
class BitMap;
^~~~~
struct

[ ] tools=yes target=debug in drivers/unix/semaphore_posix.cpp (more details in #24418)

drivers/unix/semaphore_posix.cpp:60:2: warning: 'sem_getvalue' is deprecated [-Wdeprecated-declarations]
        sem_getvalue(&sem, &val);
        ^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include/sys/semaphore.h:54:56: note: 'sem_getvalue' has been explicitly
      marked deprecated here
int sem_getvalue(sem_t * __restrict, int * __restrict) __deprecated;
                                                       ^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include/sys/cdefs.h:176:40: note: expanded from macro '__deprecated'
#define __deprecated    __attribute__((deprecated))
                                       ^
drivers/unix/semaphore_posix.cpp:77:10: warning: 'sem_init' is deprecated [-Wdeprecated-declarations]
        int r = sem_init(&sem, 0, 0);
                ^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include/sys/semaphore.h:55:42: note: 'sem_init' has been explicitly
      marked deprecated here
int sem_init(sem_t *, int, unsigned int) __deprecated;
                                         ^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include/sys/cdefs.h:176:40: note: expanded from macro '__deprecated'
#define __deprecated    __attribute__((deprecated))
                                       ^
drivers/unix/semaphore_posix.cpp:84:2: warning: 'sem_destroy' is deprecated [-Wdeprecated-declarations]
        sem_destroy(&sem);
        ^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include/sys/semaphore.h:53:26: note: 'sem_destroy' has been explicitly
      marked deprecated here
int sem_destroy(sem_t *) __deprecated;
                         ^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include/sys/cdefs.h:176:40: note: expanded from macro '__deprecated'
#define __deprecated    __attribute__((deprecated))
                                       ^

First one is pretty obvious, just a name conflict.
POSIX semaphores probably should be replaced with Grand Central Dispatch equivalents.

macOS and iOS already have own semaphore implemented and used by default, we should be able to simply remove semaphore_posix.cpp/.h and SemaphorePosix::make_default(); line from the os_unix.cpp for the macOS/iOS build without any additional changes.

First one is pretty obvious, just a name conflict.

How should it be solved? Should we rename BitMap, or do a hack such as this?
#if defined(OSX_ENABLED)
#undef BitMap
#endif

macOS and iOS already have own semaphore implemented and used by default, we should be able to simply remove semaphore_posix.cpp/.h and SemaphorePosix::make_default(); line from the os_unix.cpp for the macOS/iOS build without any additional changes.

Can you check if that works and if so PR the changes? It would be nice to build macOS with werror=yes on Travis.


PreTag Test Automation Basic Code clean-up needed

tests/test_fan.robot

*** Settings ***
Documentation This testsuite is for testing fan interface for openbmc
Suite Teardown Delete All Sessions
Resource ../lib/rest_client.robot
*** Test Cases ***
~
~
Shouldn't leave it empty with *** Test cases ****
[ ERROR ] Suite 'Test Fan' contains no tests.

tests/security/test_ssl.robot

Remove : Library RequestsLibrary.RequestsKeywords
The Library RequestsLibrary.RequestsKeywords is already include in the lib/rest_client.robot so no need to add redundant code here

tests/test_association.robot
Remove : Library BuiltIn
The built In doesn't need to be explicitly included

Probably a good idea if we could do this two combo in Test Suite Setup
Clear all logs
Open Connection And Log In

tests/test_bootpolicy.robot

Remove :
${SYSTEM_SHUTDOWN_TIME} ${5}
${WAIT_FOR_POWER_OPERATION} 30sec
This variable is not used anywhere in this suite, probably a copy paste left over code.

tests/test_eventlog.robot

Remove : Library BuiltIn
The built In doesn't need to be explicitly included
Typo : making NOT makeing in test case "makeing new log after obmc-phosphor-event.service restart"
Suggestion : Example, its good to verify the execute command to stop continue and fail latter if the command returns errors
${output}= Execute Command /sbin/reboot
To
${output} ${stderr}= Execute Command /sbin/reboot return_stderr=True
Should Be Empty ${stderr}
This would save time and error catching more better
Do for all the execute command in the code

Will fix this as well using this ticket.
PreTag test_power_restore.robot': Non-existing setting '[Documentation]' #8

Code pushed for review


DEPR: deprecate datareader in 0.17.0

xref https://github.com/pydata/pandas/pull/10061
closes #8961
point to pandas-datareader package

Closed by #10870


Dated info, link update needed

What section and page of the Curriculum is this on
ex: IPFS: The IPFS Gateways
Describe the bug

Under "Launchpad Principles", "Improve as we go" #pl-bootcamp-v1 is referenced but seems to be archived, or renamed to #pl-launchpad. Is this channel by invitation only, or has this information changed?
Under "Launchpad Principles", "Async first", the link here links to the page its currently on. Since there are a few articles on async work, which should this be updated to?

If it is additional content or a large change, create a curriculum feature request
If applicable, add screenshots to help explain your problem

Thanks @asilinphoto those are fixed, removed them both! https://protocol-labs.gitbook.io/launchpad-curriculum/about-launchpad


Feat: Enchancing the Design and Styling of the Cart Page

Describe the feature
@SUGAM-ARORA I want to make changes to the Cart Page in terms of it s styling and design. Kindly assign me this issue.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Ojas-Arora Issue assigned to you


[feature] Write data to file on SIGINT unless specified

Allows better debugging of applications that have handles that are preventing them from closing.

Thanks. You should document this. I guess it is not easy to test with the my odd testrunner, but at the very least you can have the script signal itself.
// one of these
process.argv.push('--raw-sigint');
process.env.DPROF_RAW_SIGINT = '';
require('../../dprof.js');

setInterval(function () {}, 1000):
setTimeout(function () {
  process.kill(process.pid, 'SIGINT');
}, 200);

to make the test work:

add the script file (e.g. sigint-env.json) under scripts
add an empty json file (e.g. sigint-env.json) under expected
edit update to true in test/test.js (and back when your done).
check the output manually
check the output visually using node test/show.js sigint-env.

@AndreasMadsen updated

@AndreasMadsen yup, good catch. Fixed I think.

Hmmm, the test doesn't seem to fail with either the flag or variable uncommented, not really sure what's up.

Hmmm, the test doesn't seem to fail with either the flag or variable uncommented, not really sure what's up.

process.on('exit') is called when process.exit() is called from SIGINT causing two dumps.
process.env.DPROF_NO_SIGINT = '' is falsy, we need to use !process.env.hasOwnProperty('DPROF_NO_SIGINT')
if either --dprof-no-sigint or DPROF_NO_SIGINT exists, the if statement shouldn't run. This means that || should be &&. (De Morgan's laws in formal logic :p).

I've also updated the test running to be more sensitive to some of these issues and robust in others. There are some conflicts, I think you can just drop 8fc3cb77ee63a84af75cd1f32f06098c3ed4537d and create a new one.

process.on('exit') is called when process.exit() is called from SIGINT causing two dumps.

Arrgh, right. I did some tracing of where the current node behavior comes from, and found this old issue. Apparently generically calling exit handles is a bad idea. I'm not actually sure how to exit without calling them... process.abort() could do but that produces a core file if possible...

Arrgh, right. I did some tracing of where the current node behavior comes from, and found this old issue. Apparently generically calling exit handles is a bad idea. I'm not actually sure how to exit without calling them... process.abort() could do but that produces a core file if possible...

"some digging", wow that is very old issue :) A crazy idea is to remove all SIGINT handlers and have the process SIGINT itself. But I'm also fine with just calling process.exit().

Some things:
Note sure why Signal says NaN for these:

Also, should we disable asyncHook around our own Signal handler so that it doesn't show up?

Note sure why Signal says NaN for these.

Thought I fixed that. The wait part is because the destroy property is null (happens when the destroy doesn't fire). The solution is to use root.total time instead when destroy: null (code:  https://github.com/AndreasMadsen/dprof/blob/master/visualizer/info.js#L40). It is not related to this PR so lets fix that separately, I would like this to land.
I don't know why callback is NaN too.

Also, should we disable asyncHook around our own Signal handler so that it doesn't show up?

Was, thinking about that too. If Signal handler isn't reused by another on('SIGINT') then its safe. But if it is reused then we are removing some information from the user. It may not be a big problem, as the user can pass --dprof-no-sigint for full transparency. What do you think?

The Signal handling code is at https://github.com/nodejs/node/blob/master/lib/internal/process.js#L180-L219
tl;dr: they are shared.
Maybe we should just go with this for now then?

The Signal handling code is at https://github.com/nodejs/node/blob/master/lib/internal/process.js#L180-L219

Thanks for looking that up.

Maybe we should just go with this for now then?

Definitely. Landed in<PHONE_NUMBER>561219281dd79e7fdf2303b152d5cf ee3ee189b54a1db518a1ba8944188314e490992c – Note I rename the test from sigint-env to just sigint.


question about inference

Thank you very much for releasing the code for your work.
I want to confirm that split=true  in dataset.py means that when segment_size is defaulted to 32000, will it randomly select two seconds from the audio file for training?
I set split=true when implement both train and validation, so it can be executed successfully. However,when I implement inference.py, I find that the GPU memory of my computer is not enough for inference.
Is there any way I can resolve this problem?

Yes, our code randomly selects 2-second audio segments for training purposes, while during validation, it retains the complete audio.
If you also slice the audio during validation, it doesn't affect the training process.
However, the metrics you observe on the validation set at this point may not be the complete metrics.
Additionally, when making inferences, we recommend using the complete audio segment.
If your GPU memory is insufficient, you can slice the long audio into segments for inference and then concatenate the obtained audio segments together.
But we cannot guarantee that the final metrics will be exactly the same as before.


iPhone X: Drawer content should have additional padding in horizontal layouts.

Steps to Reproduce
On the iPhone X, drawer content tiles should be inset with additional left/right padding (for LTR/RTL directionalities respectively) to account for the sensor housing notch.
See Apple's Human Interface Guidelines for iPhone X.
Dependent on issue #12098. Get at this via MediaQuery.of().padding.
Example:

Flutter Doctor
[✓] Flutter (on Mac OS X 10.12.6 16G29, locale en-AU, channel master)
    • Flutter at /Users/cbracken/src/flutter/flutter
    • Framework revision dd7e313317 (51 minutes ago), 2017-09-14 12:28:21 -0700
    • Engine revision 57a1445a45
    • Tools Dart version 1.25.0-dev.11.0

[✓] Android toolchain - develop for Android devices (Android SDK 25.0.3)
    • Android SDK at /Users/cbracken/Library/Android/sdk
    • Platform android-25, build-tools 25.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] iOS toolchain - develop for iOS devices (Xcode 9.0)
    • Xcode at /Applications/Xcode9.0.app/Contents/Developer
    • Xcode 9.0, Build version 9A235
    • ios-deploy 1.9.2
    • CocoaPods version 1.3.1

Drawer items are padded as of #13545. Work is still required for the heading area.

I think there's either still a remaining issue with this or (more likely) I don't know what I'm doing. Here's a screenshot of my app:

The white area above the blue should also be blue.
The structure is

Drawer

ListView

Container with topCenter alignment and BoxDecoration for the dark blue background

DrawerHeader

CachedNetworkImage

ListTile
ListTile
…

@jonahgreenthal Please create a new issue. This one is long closed.


Improving the UI of 'Add Books' page

Description
Hi
While exploring the website, I noticed that the 'Add Books' page looks simple and unappealing.
Being a GSSoC'23 Contributor, I would like to improve the UI of the 'Add Books' page.
Plan
As per my plan, I would like to do the following tasks to the page:

Making the page look more attractive
Adding drag and drop file functionality to the Upload Books section
Adding responsiveness to the page

So, can you please assign this work to me under GSSoC'23 ?
Possible Implementation
Using HTML, CSS, JavaScript and React
Checklist

[x] I have checked the existing issues.
[x] I have read the "Contributing Guidelines".
[x] I am willing to work on this issue.
[x] I am a GSSoC'23 contributor.

@dhingra-madhav Go for it


Upgrade CLI hash_validator dependency to 0.8.0

Upgrade hash_validator dependency from ~> 0.7.1 to 0.8.0.

None of the specs seem related to hashes and the validation part is quite well covered. Rebased anyway.

Merge conflict on #3208

Confict be gone


The app tries to turn down my keyboard light instead

Hello! Thank you for sharing this helpful script! I'm using Macbook Air M2 with the latest Sonoma 14.1.1. When I successfully installed the script as app and run, it seems to click the wrong button that turns down my keyboard light instead. I suspect that the problem lies in:
# "Keyboard Shortcuts..." Button
set keyboardButton to 1
click button keyboardButton of group 2 of scroll area 1 of group 1 of group 2 of splitter group 1 of group 1 of window 1

Where the keyboardButton refers to a wrong button. However, I'm not familiar with the grammar of the apple script. So it would be great if you could elaborate a little more on what's keyboardButton and potential ways to fix the script. Thanks!

Hello! Thanks so much for writing in, and glad you found the script useful.
You're not the first person to come across this issue so I've looked into the code history and can you try something for me?
In the section of the script you posted, try to change the 1 to a 3 - as in the code below?
# "Keyboard Shortcuts..." Button
set keyboardButton to 3

… Try that for me and write back. If it works, then I'll make a comment in the code as you're the third person to write about this! Thanks.

After replacing 1 to 3 as suggested the script seems to function properly. Thank you for your help!

I had the same issue on an M1 Air running Sonoma 14.0.
Changing the value from 1 to 3 fixes it.

I've now updated the script to actually try button 3 (in a try/catch) then button 1, so no-one else should ever have this problem again. (And thanks to @cartesian-plane this might even go away entirely with v14 thanks to new reveal anchor code which I'd not seen before). Closing.


⚠ Bofrost.es has degraded performance

In 2088a1b, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1579 ms

Resolved: Bofrost.es performance has improved in 337c528 after 7 minutes.


Failing to parse auto-generated function signature

Version and Platform (required):

Binary Ninja Version: 4.0.4958, ddff9339
OS: macos
OS Version: 14.4
CPU Architecture: arm64

BN internal database: major dine favor
Bug Description:
00012d5c          x0_3, v0_1, v1_1, v2_1, v3_1, v4_1, v5_1, v6_1, v7_1 = DjiUser_FillInUserInfo(userInfo: &userInfo)
00012d6c          if (x0_3 != 0)

The Change Type dialog fails to parse its own type spec. It puts the structure definition in the signature
long unsigned int DjiUser_FillInUserInfo(struct { char appName[0x20]; char appId[0x10]; char appKey[0x20]; char appLicense[0x200]; char developerAccount[0x40]; char baudRate[0x7]; }* userInfo)

Related to #5325.

Related to https://github.com/Vector35/binaryninja-api/issues/5328

We probably need to better understand https://github.com/Vector35/binaryninja-api/issues/5328 to assign a component tag to this issue


Add frontend for search

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#146  👈
#145
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @illarionov and the rest of your teammates on  Graphite


Production-ready?

Apologies that this more of a question than an issue, but I found this repo thanks to @nickhammond's comment in a Rails issue and would love to it a try.
But I was wondering you're currently using it in production, or if it's still in an experimental stage? Any rough edges that would be good to know about?
In any case, thanks so much for working on this - it's definitely needed!

Yea this is ready to roll and im using it in production currently. Wrapping up some of the api and prepping to cut v1.

Party, congrats @npezza93!

@npezza93 If you run the same benchmarks with litestack does the litedb adapter perform any better than native sqlite3?

@npezza93 Also, since solid_cable is DB agnostic it'd be great to run these against mysql/postgres. I can help with this on Digital Ocean and a local mysql/postgres instance. I'll use what I can from your benchmark comments and run this tomorrow. I'll try and run this as a single file Rails app so we can re-use the same setup too https://greg.molnar.io/blog/a-single-file-rails-application/

Cool. I wouldnt bother with postgres since it has it's own dedicated action cable adapter.

As for litestack:
I ran sqlite3 with the patches described here: https://github.com/rails/rails/pull/50371#issuecomment-2145725946 and against rails main so it should be virtually identical. Didnt end up with any read locks or anything

@npezza93 Well the current pg adapter has size limitations for what can be sent over the wire, solid_cable would be a great replacement for those limitations.

@npezza93 Heard on sqlite3/litestack. That's a great thread, thanks for sharing.

Oh interesting. How big is the limitation?

8k so for simple over the wire and views it's fine but it's sort of unpredictable because you have to keep that in mind when building an app with hotwire.

Yea this is ready to roll and im using it in production currently. Wrapping up some of the api and prepping to cut v1.

That's great news - thanks so much!


Added Russian translation.

Adds ru- locale to the mod.

Hi! Sorry, I hadn't seen this. Would you be able to also create the translation for the extended train setting that I just implemented in 0.3.4? Thanks so much!!

Actually its all fine

Will just take it for now to get 0.3.5 out there

Hello! Translated the extended train setting as you've asked.


CrunchbaseAPIException

Hi Fahad,
nice work on this library!
When running one of your examples ("Entities API") I get the following error:

Exception has occurred: CrunchbaseAPIException
Expecting value: line 1 column 1 (char 0)
StopIteration: 0
During handling of the above exception, another exception occurred:
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
During handling of the above exception, another exception occurred:
requests.exceptions.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
The above exception was the direct cause of the following exception:
File "C:...\crunchbase_library_test.py", line 13, in
entity = org_api.get(
py_crunchbase.apis.base.CrunchbaseAPIException: Expecting value: line 1 column 1 (char 0)

Not sure exactly what is causing this but seems to be related to the formatting of the JSON request.
I am using requests version 2.28.2 which meets the requiement value set in requirements.txt
Best Regards,
Sebastian

Hi @Sebastian-ehrig. Thanks for bringing it up. I've a few meetings in queue. I will try and look into it tomorrow.

Hi @FahadNoor thanks for your quick response.
Here is the code:
`from py_crunchbase import PyCrunchbase, Cards
import requests
print(requests.version)
API_KEY = '***************'
pycb = PyCrunchbase(API_KEY)
org_api = pycb.organizations_api()
cards = Cards.Organization
entity = org_api.get(
entity_id='tesla-motors',
field_ids=['website', 'facebook', 'categories', 'short_description', 'founded_on', 'rank_org_company'],
card_ids=[cards.founders, cards.raised_funding_rounds]
)
print(entity.website, entity.cards)`

Thanks. Can you check if you are using the correct version of requests module? Because version attribute isn't available in 2.28.2. It might have been available in the older versions.

Hi @FahadNoor, that must have been a formatting error when copying the code into the text-field. I was indeed using in my code "requests.version". The requests version that I am using is 2.28.2.

ah, that makes sense. Unfortunately I still haven't been able to reproduce the issue. I've tried with basic and enterprise API key.
Can you run the code through debugger and look at the response/data you are getting from Crunbhase? You should be able to check the value of response and data here.

The response data returns: <Response [404]>

It might be an API key related issue. Can you try with a different key?

I don't think it is the API key. I have made a quick test with some simple code which returns 200.
`import requests
api_key = '************************'
base_url = "https://api.crunchbase.com/api/v4/entities/"
company_endpoint = "organizations/"
company_name = "tesla-motors"
query = {
"card_ids": [
"founders",
"raised_funding_rounds"
],
"field_ids": [
'website',
'facebook',
'categories',
'short_description',
'founded_on',
'rank_org_company'
],
}
Build the full API request URL
url = base_url + company_endpoint + company_name + "?user_key=" + api_key
Send the API request and get the response
response = requests.get(url, params = api_key , json = query)
Check the status code of the response to make sure it was successful
print('Response: ', response.status_code)`

Hi @FahadNoor , I have tried your examples again on a Linux system and there it is working fine! This is perhaps an issue related to windows.

oh, weird. I will give it a try on Windows. Thanks for sharing this info.

@FahadNoor I have the same problem... Could it be because the Starter level of access is being used?
I'll ask here at the same time - do you think there is a way to get data from Signal & News -> Timeline via API?

Hi @alex-tarasov-6669, this seems to be an issue related to Windows. I haven't got a chance to test it on Windows myself. I will try next week. Can you try on Linux?
As for Signal & News, I am not sure. I would suggest to explore the organisations API with available cards like ipos, layoffs, event_appearances etc and see if it works for you.

@FahadNoor Just tested it on my GCP instance and get: py_crunchbase.apis.base.CrunchbaseAPIException: MD403:  insufficient permissions to read field categories. MD403:  insufficient permissions to search field funding_total
So, probably I may be right about permissions and access level.
`from py_crunchbase import PyCrunchbase, Entities
from py_crunchbase.apis.search.predicates import Currency
pycb = PyCrunchbase('key')
api = pycb.search_organizations_api()
org_facet_ids = Entities.Organization.Facets
api.select(
'identifier', 'categories', 'location_identifiers', 'short_description', 'rank_org'
).where(
funding_total__between=[Currency(25000000), Currency(100000000)],
location_identifiers__includes=['6106f5dc-823e-5da8-40d7-51612c0b2c4e'],
facet_ids__includes=[org_facet_ids.company]
).order_by(
'rank_org'
)
for page in api.iterate():
for company in page:
print(company.permalink)`

Oh, when you said that you had the same issue, I assumed it was same as Sebastian. He was getting a 404 on Windows.
But yes you are right, it seems like an API access/permission issue.

@Sebastian-ehrig I've fixed this issue in the latest release. I am closing the issue. Feel free to reopen it if you still face the same problem.


Date formatting for downloads

Hi @john-friedman,
I have taken care of issue - https://github.com/john-friedman/datamule-python/issues/11
Heres an example
downloader = dm.Downloader()

downloader.download(form='10-K', ticker=['TSLA'], output_dir='filings')

filings downloaded -

Hi @AnirudhJM24, thank you for the pull request. I ended up integrating your code manually (in a bit of a rush, and there was an issue with previous form d commits that I did not want to integrate yet). I've added you to the contributors.md

(To be clear this is because I don't know how git works w.r.t. approving some modifications but not others, this is something I need to learn, and I plan to this weekend)

(To be clear this is because I don't know how git works w.r.t. approving some modifications but not others, this is something I need to learn, and I plan to this weekend)

Hi @john-friedman
No issues, it was a mistake from my end, I created a branch from the old PR branch


Add UXSD_INVALID to capnp to distinguish unset enumerations.

These changes were required to make https://github.com/SymbiFlow/vtr-verilog-to-routing/pull/340 work with the capnp rr graph format.

I think trying to merge this over a flaky connection resulted in a github bug - the merge commit is done but the PR is not closed.


Add continuous integration

With this I'd like to build cmdc for each platform, multiplied by each version of Maya. I think what needs to happen is an independent build for every service pack of Maya, since they introduce new members to the API in each one. :(
Each Maya version has 3-4 service packs, and with 4 supported versions of Maya, and 3 supported platforms, that comes in at..
4 * 4 * 3 = 48

That's right, 48 builds for every commit made to this repository. A number that will grow along with new versions of Maya being released, but shrink as we deprecate support for those ~5+ years old.
This first PR currently includes a single version of Windows, that I intend to expand upon as I get things running. Iterating on CI scripts is a PITA, taking 5 mins each time just to see if the syntax of your build script is even correct. So, there will be many commits, but in this case I expect I'll squash the commits during merge since the history of my failures are of no relevance to future code-archeologists.
I've added a build-script for CI, that I expect we'll be able to use locally as well to ensure an identical environment. It was intended to replace the setup.py script as I expected it to be significantly faster. But! As it turns out, the bottleneck was not Python, but pybind11. It still takes roughly 5 seconds to build. This number is expected to grow significantly as we start adding to the binding.. So odds are we'll split translation units apart for parallel builds and to also iterate on a subset of the binding at once. Cross that bridge when we get to it.
$ build_win32.ps1
(1) Parsing..
(1) Finished in 40 ms
(1) ----------------------------
(2) Compiling..
main.cpp
(2) Finished in 5119 ms
(2) ----------------------------
(3) Linking..
   Creating library C:\github\mottosso\cmdc\build\cmdc.lib and object C:\github\mottosso\cmdc\build\cmdc.exp
(3) Finished in 221 ms
(3) ----------------------------
(4) Cleaning..
(4) Finished in 44 ms
(4) ----------------------------
Successfully created .\build\cmdc.pyd in 5424 ms

PyPI
Once builds work on all platforms and versions, I'll look into distributing cmdc on PyPI. The crux here is..

How will the user install cmdc?

You might think mayapy -m pip install would do, but where would that be installed? If we put it under c:\program files\autodesk\maya2020\python\site-packages then that's great. But which Maya 2020? 2020.3? Or 2020.4?
Furthermore, you wouldn't want to pollute your system Maya install directory, so how about c:\users\marcus\Documents\maya\2020\scripts? Sure, but the problem remains, 2020.3 won't work with 2020.2.
And how would you even specify which version of cmdc you want?
$ pip install cmdc[maya2020.3]

Could be one option.. Absolute worst case, we simply won't distribute it on PyPI and have people download binaries it off of GitHub directly. Cross that bridge when we get to it.
As a final option off the top of my head, we could also distribute it as a Maya "module", and ship all versions of cmdc as-one. Not sure if those include support for dealing with service pack differences however.

Ok, one platform at a time.


Add further requirements to execution environments

Is your feature request related to a problem? Please describe.
I want to import the following libraries in the execution environments:
langdetect==1.0.9
phonenumbers==8.13.0
python-levenshtein==0.20.8
textblob==0.17.1
textstat==0.7.3

Describe the solution you'd like
Add the requirements to the exec-env parent image

langdetect==1.0.9
nltk==3.7
phonenumbers==8.13.0
python-levenshtein==0.20.8
textblob==0.17.1
textstat==0.7.3
translate==3.6.1
spacy==3.4.2
quantulum3==0.7.11
LeXmo==0.1.4

fuzzywuzzy==0.18.0

current requirements included. for new one reopen the issue

better_profanity==0.7.0 (will mention in the readme that the module using this will work in refinery versions > 1.6)

stemming==1.0.1

nltk.downloader words stopwords wordnet omw-1.4 brown punkt

current requirements included. for new one reopen the issue

bayes_opt==1.4.2

google-search-api==2.4.1

included in image 1.8.0 reopen with new requirements

textacy==0.12.0

included in image 1.8.1 reopen with new requirements

scikit-optimize==0.9.0

included in image 1.8.2 reopen with new requirements

holidays==0.21.13

sumy==0.11.0

Please update OpenAI from openai==0.25.0 to openai==0.27.7. Otherwise, GPT-3.5-Turbo and GPT-4 won't be usable in refinery. Thanks! :)

included in exec env image 1.11.0 reopen with new requirements

Please include knowledge==0.3

PyPDF2==3.0.1

@SvenjaKern I added pypdf==3.15.5 instead. pypdf and pypdf2 have the same maintainer and he wants the community to switch to pypdf (https://stackoverflow.com/questions/63199763/maintained-alternatives-to-pypdf2, first answer). He states that both should be similar. PyPDF2 also raised a dependabot alert, which also suggested the switch to pypdf as fix.
Please check if the new brick also works with pypdf. If that's the case, you can simply close the issue.
Otherwise, I will add pypdf2 instead.

@FelixKirschKern Please add tiktoken==0.4.0 and remove pypdf==3.15.5 if it was added, as it's not needed. Thanks!

also increase psycopg2-binary version to psycopg2-binary==2.9.7

Added tiktoken==0.4.0 and removed pypdf==3.15.5, updated urllib3

increase psycopg2-binary version to psycopg2-binary==2.9.7 (or maybe even higher)

Updated psycopg2-binary and added language support for japanese


[Build] Pass object files via a file to Swift compiler

This is required for packages with large number of dependencies which
may cause the command line invocation to surpass the arg max limit.
rdar://problem/38803919 Unable to compile the code via terminal using swift test

@jrose-apple wohoo, it works!

@swift-ci please smoke test

Unfortunately this seems to be breaking Swift's integration tests on Linux (and thus blocking merging via smoke tests), possibly due to not handling spaces properly:
https://ci.swift.org/job/swift-PR-Linux-smoke-test/7840/consoleFull#13225944293122a513-f36a-4c87-8ed7-cbc36a1ec144
00:28:57 /home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@2/branch-master/buildbot_linux/none-swift_package_sandbox_linux-x86_64/usr/bin/swift-build-tool -f<EMAIL_ADDRESS>spaces/special tool/.build/debug.yaml' main -v
00:28:57 /home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@2/branch-master/buildbot_linux/none-swift_package_sandbox_linux-x86_64/usr/bin/swiftc -module-name special_tool -incremental -emit-dependencies -emit-module -emit-module-path<EMAIL_ADDRESS>spaces/special tool/.build/x86_64-unknown-linux/debug/special_tool.swiftmodule" -output-file-map<EMAIL_ADDRESS>spaces/special tool/.build/x86_64-unknown-linux/debug/special_tool.build/output-file-map.json" -c<EMAIL_ADDRESS>spaces/special tool/main.swift"<EMAIL_ADDRESS>spaces/special tool/some file.swift" -I<EMAIL_ADDRESS>spaces/special tool/.build/x86_64-unknown-linux/debug" -swift-version 3 -target x86_64-unknown-linux -sdk / -Onone -g -enable-testing -j32 -DSWIFT_PACKAGE -DDEBUG -module-cache-path<EMAIL_ADDRESS>spaces/special tool/.build/x86_64-unknown-linux/debug/ModuleCache"
00:28:57 /home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@2/branch-master/buildbot_linux/none-swift_package_sandbox_linux-x86_64/usr/bin/swiftc -target x86_64-unknown-linux -sdk / -g -L<EMAIL_ADDRESS>spaces/special tool/.build/x86_64-unknown-linux/debug' -o<EMAIL_ADDRESS>spaces/special tool/.build/x86_64-unknown-linux/debug/special tool' -module-name 'special tool' -emit-executable -Xlinker '-rpath=$ORIGIN' '<EMAIL_ADDRESS>spaces/special tool/.build/x86_64-unknown-linux/debug/special tool.product/Objects.LinkFileList'
00:28:57
<unknown>:0: error: no such file or directory<EMAIL_ADDRESS>00:28:57 <unknown>:0: error: no such file or directory: 'spaces/special'
00:28:57 <unknown>:0: error: no such file or directory: 'tool/.build/x86_64-unknown-linux/debug/special_tool.build/main.swift.o'
00:28:57
<unknown>:0: error: no such file or directory<EMAIL_ADDRESS>00:28:57 <unknown>:0: error: no such file or directory: 'spaces/special'
00:28:57 <unknown>:0: error: no such file or directory: 'tool/.build/x86_64-unknown-linux/debug/special_tool.build/some'
00:28:57 <unknown>:0: error: no such file or directory: 'file.swift.o'
00:28:57 error: terminated(1): /home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@2/branch-master/buildbot_linux/none-swift_package_sandbox_linux-x86_64/usr/bin/swift-build-tool -f<EMAIL_ADDRESS>spaces/special tool/.build/debug.yaml' main -v output:

Opened https://github.com/apple/swift-package-manager/pull/1624

Thanks; hopefully the proper fix is easy too.

*sigh* I'm sorry, I got the file format wrong. It's whitespace-separated, not newline-separated, like a shell script. You can use quotes or backslashes to escape whitespace in paths.

Second attempt: https://github.com/apple/swift-package-manager/pull/1626


Organize XmlNode vs. XmlFile types

I hope you will find this much more organized and easier to work with. It's organized in the following manner:
XmlFile classes for dealing with the different Xccdf XML files :

DatastreamFile
TestResultFile # this used to be OpenscapParser::Base
TailoringFile

XmlNode classes which match a capitalized Xccdf type in XML (i.e. <Profile /> not <profile />):

Benchmark
Profile
Rule
RuleIdentifier
RuleReference
TestResult
RuleResult
Tailoring

Modules for parsing the various XmlNode types from XmlFile types:

Benchmarks
Profiles
Rules
RuleReferences
TestResults
RuleResults
Tailorings

Signed-off-by: Andrew Kofink<EMAIL_ADDRESS>

Rebased


[XLA] Add F16 as type for the literal conversion functions

The F16 (synonym for eigen half) type was not included in the literal conversion functions.  float/double is supported as a type conversion by the eigen half, so adding identical lines as the native types.

Can one of the admins verify this patch?

Jenkins, test this please.

mm.  that's odd.  i think our buildbot runs that test.  I'll double check.

i see.  we run tensorflow/compiler/xla/tests..., and not tensorflow/compiler/xla:...
will fix asap.

fixed.

Jenkins, test this please.


As a developer, I want to have the mockup/wireframe of the web page for starting the implementation

estimate 5

@MrDecipher , @NancyLakhanpal : please have a look to the wireframe from draw.io. I dont create new branch for this issue because it is a image from outside source code. If you approved, just add a comment under this issue. Then its good to go.

@MrDecipher , @NancyLakhanpal : please have a look to the wireframe from draw.io. I dont create new branch for this issue because it is a image from outside source code. If you approved, just add a comment under this issue. Then its good to go.

@tranvnb Approved from my side.

@tranvnb Approved from my side.


Could not find compiled dynamic library error.

Description
when i log into neovim it gives me this error
[cord.nvim] Could not find the compiled dynamic library at: C:/Users/User/AppData/Local/nvim-data/lazy/cord.nvim/cord.dll. Please re-run the build script

Operating system
Windows 10 Pro 22H2
Neovim version
NVIM v0.10.0
Build type: Release
LuaJIT 2.1.1713484068

Steps to reproduce

nvim -nu ./init.lua
thats it you got error'd

Expected behavior
for rpc ro work
Actual behavior
error and rpc doesnt appear
Minimal configuration
-- Bootstrap lazy.nvim
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
if not (vim.uv or vim.loop).fs_stat(lazypath) then
	local lazyrepo = "https://github.com/folke/lazy.nvim.git"
	local out = vim.fn.system({ "git", "clone", "--filter=blob:none", "--branch=stable", lazyrepo, lazypath })
	if vim.v.shell_error ~= 0 then
		vim.api.nvim_echo({
			{ "Failed to clone lazy.nvim:\n", "ErrorMsg" },
			{ out, "WarningMsg" },
			{ "\nPress any key to exit..." },
		}, true, {})
		vim.fn.getchar()
		os.exit(1)
	end
end
vim.opt.rtp:prepend(lazypath)

-- Make sure to setup `mapleader` and `maplocalleader` before
-- loading lazy.nvim so that mappings are correct.
-- This is also a good place to setup other settings (vim.opt)
vim.g.mapleader = " "
vim.g.maplocalleader = "\\"

-- Setup lazy.nvim
require("lazy").setup({
	spec = {
		-- add your plugins here
		{
			"vyfor/cord.nvim",
			build = ".\\build",
			event = "VeryLazy",
			opts = {},
		},
	},
	-- Configure any other settings here. See the documentation for more details.
	-- colorscheme that will be used when installing plugins.
	install = { colorscheme = { "habamax" } },
	-- automatically check for plugin updates
	checker = { enabled = true },
})

Hey! Can you please try to reinstall the plugin, and if that doesn't help, try manually building it and see if there are any build errors:
cd %LOCALAPPDATA%\nvim-data\lazy\cord.nvim && call build.bat

where should the build appear?
> call build.bat

rustc --crate-type=cdylib -C opt-level=3 -C panic=abort -C lto=fat -o cord src/lib.r

this is what i got after running call build.bat in that directory

That should create a file named cord in the same directory. cord.nvim will then append a platform-specific file extension (in your case, .dll).

Is it able to locate the file now?

yes, its working! thank you. though i wonder why it would only work after i manually built

I suppose you initially used build = "./build" in lazy, which fails to locate build.bat in the command prompt (default shell on neovim, though it works in powershell). And after changing it to .\\build, lazy won’t rebuild unless you reinstall or update the plugin, so either that or manual build.


RDP to AAD-joined PC from non-joined PC

I'm looking for a couple things related to RDPing to an AAD-joined remote computer from a local computer that is not joined to AAD or any domain:

https://morgansimonsen.com/2015/11/06/connecting-to-an-azure-ad-joined-machine-with-remote-desktop/ has pretty old instructions about RDPing to an AAD-joined machine. It looks like things have improved since then, except for the scenario where the local computer is not AAD-joined. In that case, those instructions appear to still be the only way you can make it work. It requires disabling the NLA requirement on the remote machine and setting enablecredsspsupport:i:0 in the RDP file. Can anyone confirm that this is still the only way to connect in this scenario? I'd prefer to re-enable NLA if I could.
Either way, can content be added to this page addressing this scenario that either explains how to do it or declares that it's unsupported?

#3866 had the same topic as this but was closed without a resolution.

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

ID: 72e1cc18-3fc2-d7fa-7ff3-20fc9a3596d4
Version Independent ID: 8ed24664-ce8c-7c08-654a-d3ef267f274f
Content: Connect to remote Azure Active Directory-joined PC (Windows 10)
Content Source: windows/client-management/connect-to-remote-aadj-pc.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@officedocsbot assign @jvsam

Hi @Dansimp I believe you own this topic so I am tagging you here. Kindly see @nickwalkmsft's feedback. Is this something that can be incorporated to this doc? Or do you want me to ask someone from the team to create a pull request to include the suggestions? Looking forward to hearing from you. Thank you.

Either way, can content be added to this page addressing this scenario that either explains how to do it or declares that it's unsupported?

Hello again @Dansimp I just want to follow-up if you had a chance to look into this and if changes need to be made on the content as per the information provided? Thank you.

I think it is very important to add the scenario described in the first post. Here is also a good description.
Especially the part if you want to join from WORKGROUP computer.
https://www.niallbrady.com/2017/08/23/how-can-i-rdp-to-an-azure-ad-joined-windows-10-device/

Thank you for sharing that reference link @celeroll. This is currently under review and we will update the status of this issue soon.

It seems the initial setup of windows has an impact on the ability to use RDP. I have 2 machines, once setup exclusively using AAD (machine 1) and the other one added later on to the Directory (machine 2).
I can RDP, using my AAD logins, from machine 1 to machine 2 without any issues, but the other way around requires the little hacks mentioned earlier. I looked into Windows Defender Credential Guard but it's not turned on.

Hello again @nickwalkmsft, thank you for your patience. Please be informed that the suggested updates to this documentation as per your feedback (See pull request #5344) have already been merged. Please expect the changes to be displayed here at docs.microsoft.com in the next scheduled publishing run.
Again, thank you for your feedback!

Hi @CpBobette, we believe in your case, it will be best to contact Windows 10 Support to get immediate assistance. There are also other support resources that you can explore like the Microsoft Tech Community or the Technet forums where you can post your question and get help from the community.
Kindly keep us informed if you find any relevant information that can be incorporated into this documentation. Thanks for being part of the Microsoft Docs community!

@officedocsbot close

Any chance of actually getting a good solution to this problem? I've been searching for a way to connect to an AAD-joined PC without resorting to disabling NLA. It's great that additional security can be enforced in such cases, but the requirements are too restrictive for a lot of use cases, which is why all you can find online are blogs on how to "Connect to an Azure AD joined PC" where the first step is to flat out disable NLA as the "solution". Ok, it works if I disable NLA, but then can we get better security than that? NLA was there was a reason, it's not made to be disabled.

@awakecoding : You may want to look elsewhere for answers, unless you have suggestions for improving one or more specific pages in the MS Docs repository.
Disclaimer: I am not affiliated with Microsoft or the Microsoft Docs teams here on GitHub.
The MS Docs team members might not have time to answer your question, unless this is something they would want to add to the existing documentation.
The intention of this repository is to improve on the existing documentation by providing the users a way to report factual errors, improvement suggestions and request updates for new versions.
Try the following resources to get in touch with users who might be able to help you :

TechNet forums: https://social.technet.microsoft.com/Forums/en-US/home
MSDN: https://social.msdn.microsoft.com/Forums/en-US/home
Microsoft Community: https://answers.microsoft.com/


Parse _borrowing x in patterns as a borrow binding.

Treat it as a contextual keyword when followed by an identifier, like our other ownership-related declarations and operators.

@swift-ci Please test


dockerized

Could you please provide details as to what this solves and what you have done?

No details were provided.


Added More Languages on Publish Form

PR Checklist

Please check all that apply to this PR using "x":

[x ] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[x ] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

I added 14 Languages for now

These need to come from our protobuf values dynamically. We could probably do this round manually, but please reference the issue to make sure those values are available .
https://github.com/lbryio/lbry-desktop/issues/2455

okay,, just help user named Link on our chat room sir

@tzarebczan if the values added here are correct, i think we should include this before #2455, just to get some more languages in

this value are correct based on iso code of these countries

Thanks for checking @ykris45 ! LGTM @seanyesmunt

Thanks You Mr Thomas..

Had issues with merging this branch so I just created a new commit and pushed to master. Will be included in the next release @ykris45 !


Could not import Salesforce objects

Select a subset of the NPSP objects and two custom objects.  >100k records in 3 of the 10 object tables.  The error message returned does not aid in determining why the fetch failed.  Are there more granular logs available?
Oops, things did not go smoothly, we are unable to provision the data lake. This might be an intermittent problem. Wait for a few minutes and try again. If you continue to get this error, you can peek at the technical details and seek assistance from a solution implementor agency.
Failed to fetch

Is it possible to share a screenshot related to the error?
In general, every step has been logged using CloudWatch but if I can see a screenshot, it'd be more helpful to point to the right CloudWatch logs.

I reduced the number of objects, specifically removing Task from the extract and it was eventually successful.  The broader question is to what extent implementation teams have access to granular error logging (Field-Level Security issues, etc) that may pinpoint issues.


can not run,Provision failed with message: The device used is not whitelisted for Instant Apps deployment.. Do you want to retry

when I run hello-split use android studio 3 canary 1, it show a dialog window

"Provision failed with message: The device used is not whitelisted for Instant Apps deployment.. Do you want to retry?"

my device is nexus 5,  android version is 6.0.1 , setting -> google -> instant app has open.
Look forward to your reply. thank you

We currently don't support the Nexus 5 as a device for Android Instant Apps. An emulator with Google Play Services and a logged in user will allow you to develop as well.


Issue #823 I explained the DSA in Hindi

Fixed #823
PR Checklist:

[x ] My submission is formatted according to the guidelines in the contributing guide
[ x] My addition is on refer on the language README.md file
[ x] My addition does not have a spelling problem
[ x] My submission has a proper and user-friendly description of the algorithm
[ x] My submission has the time complexity of the algorithm
[ x] My submission has sample input-output of the program (NOT FOR PYTHON)

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] New algorithm
[ ] Optimization in previous algorithms
[ ] Code style update
[ ] Refactor
[x ] Documentation
[ ] Other, please describe:

Briefly describe the changes in this PR
It explains the DSA in Hindi

@aryaman0777 I was wondering what happened to your pull request. Did it get accepted or not. Also, 'hi' (Hindi) folder is also not visible in documentation folder. Do you know why?


Improve Windows build instruction

Description
A clear and concise description of this pull request.
There are some confusing pitfalls in Windows build insturction with Bazel. This PR improve it and make it easier for Windows users to build Mozc with Bazel.
Issue IDs
Issue and/or discussion IDs related to this pull request.

https://github.com/google/mozc/issues/932
https://github.com/bazelbuild/bazel/issues/23882

Steps to test new behaviors (if any)
A clear and concise description about how to verify new behaviors (if any).

OS: Windows 11
Steps:

Build Mozc by the steps written in docs/build_mozc_in_windows.md

Additional context
Add any other context about the pull request here.

We have merged your PR.
Thank you for the contribution!


Tutorial

Add tutorial section to docs that gives a simple, but working example of how to configure a run, execute the simulation, parse the result, and then plot it.

This should probably be a set of gallery examples instead

Examples:

Setting up the configuration dictionary (just the current contents of the config.rst page)
Actually running a simulation and parsing output
Setting up the magnetic field and gravitational profile fits


[TE] Added DataSource.getMinDateTime() API implementation for PinotDataSource

Description
The min datetime API is required to fetch the earliest available timestamp
for a given dataset in Pinot.

By default the API returns -1L for other datasources apart Pinot.
If the dataset is not present in Pinot, it returns 0
Else it queries pinot for the timestamp
PinotDataSourceMaxTime has been renamed to PinotDataSourceTimeQuery
The collectionToPrevMaxDataTimeMap map has been removed since it was being updated but never queried.

There are NO backward incompatible changes.

Thanks @akshayrai
The min date will be useful when trying to figure out how far to look back when figuring out anomalies when a new alert is created.


feat: Add geography support

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

Fixes #136 🦕

Note that I refactored the docs a bit to make navigation easier and to add api documentation for GEOGRAPHY, WKT and WKB.
I also dropped documentation for the old api for doing fry-run queries, since we decided we were going to drop that api.

I added the "do not merge" label.  We won't merge this until the after the rename branch lands.

I wonder if we want to merge this before the rename just to make the docfx session happy?

I'm working on a branch that combines the rename and geography (and snippets to be used in the geography doc).
I suggest we decide next week whether to wait for Dan to fix docfx or to include geography in alpha 1.

I merged the rename branch into this. Maybe that was a mistake.
We'll either want to merge the rename branch first, at which point this will be smaller, or we'll merge this.

Everything has been reviewed here except: https://github.com/googleapis/python-bigquery-sqlalchemy/pull/228/commits/6343bee73490b68b513698d0c9a79a99ceaa8438, which changes the geography documentation to use snippets, which found a number of bugs. :)  I also improved one of the query examples.
I tried to format the query examples to be more readable in the documentation, but black wouldn't let me. :(

@tswast I'm going to assume you still approve unless I hear otherwise. :smiley:
I'm thinking of waiting to merge this until after 1.0.0-a1 is released, unless you want me to include these changes in a1.


Cookie value not correctly checked against csrf token value

As long as a cookie and a csrf token (in header or body) exist, the verifyToken() check seems to return 'true'.
You can edit the cookie or csrf token to be of any value and the mutation request will still be sent.
We use the library as in the nextjs13-approuter-js-submission-static example. In middleware, we implement the functions as stated in the example. When making a PUT request with fetch(), we attach the token to the request body. We also tested with attaching it to the header, which seems to give the same result.
Example of cookie modification:

We tried to debug. The cookie value recieved by middleware is "test". In the verify() function hash is equal to hashCheck, which it probably shouldn't be.
We may have an implementation error or a misunderstanding of the library.
Please let me know if you need more information.

Thanks for taking such a close look at the library!
I believe the library code is working as expected because in this particular case the application's client code doesn't fetch/generate the csrf token until after you hit Submit so the csrf token used subsequently will be valid for the modified secret in the cookie:

Upon form submission, the client sends a GET /csrf-token request to the server:
https://github.com/amorey/edge-csrf/blob/main/examples/next13-approuter-js-submission-static/app/page.tsx#L13-L15))

The server responds with a new csrf token generated in realtime from whichever secret value is in the cookie:
https://github.com/amorey/edge-csrf/blob/main/examples/next13-approuter-js-submission-static/middleware.ts#L22-L25.

After obtaining a csrf token, the client makes a request to the protected endpoint at POST /form-handler with the token/cookie pair from the previous step:
https://github.com/amorey/edge-csrf/blob/main/examples/next13-approuter-js-submission-static/app/page.tsx#L17-L22

This means that the POST request will pass CSRF checks because the new token matches the secret in the modified cookie. Please note that if you were to change the secret between these steps then the library would reject the token.


Fix integration tests to include new DotNetCore aliases

Currently a few of the .NET Core CLI aliases are missing integrations tests
https://github.com/cake-build/cake/blob/develop/tests/integration/Cake.Common/Tools/DotNetCore/DotNetCoreAliases.cake
Available aliases:
https://cakebuild.net/api/Cake.Common.Tools.DotNetCore/DotNetCoreAliases/

I'd love to take a stab at this, I'm fairly new but I feel confident I could accomplish this!

If this issue is still up for grabs, I would like to help!

The last two comments are over a year old, and I have been interested in getting into contributing the open source. I went ahead and added some tests for the missing aliases and created a PR following the contribution guidelines.

fixed by #2686


Fix 'name cannot be modified' test

Without the IIFE, expect() fails to detect the failure; see https://forum.exercism.org/t/8300

This is blocked by #2321

Could you rebase this on main?


need approved label for /create command

Without a visual label like approved I cannot tell which of the issues I actually approved and they are going through the motions, versus ones I need to actually review and issue /create command, until the instance created label is affixed.

Fixed in https://github.com/MorphoCloud/MorphoCloudWorkflow/commit/7593cb98ff05fdf49f25c2942da8cd11faabd699


[Bug] "Unable to load Firebase app options" when running as standalone executable in a folder which contains non-ASCII characters

[REQUIRED] Please fill in the following fields:

Unity editor version: 2020.3.19f1
Firebase Unity SDK version: 8.6.2
Source you installed the SDK: .unitypackage
Problematic Firebase Component: App
Other Firebase Components in use: Auth
Additional SDKs you are using: None
Platform you are using the Unity editor on: Windows
Platform you are targeting: Desktop
Scripting Runtime: Mono

[REQUIRED] Please describe the issue here:
When running as a standalone executable, Firebase is unable to create app/load Firebase app options when a non-ASCII character is present in the directory name/path.
Here is a YouTube video demonstrating this bug:
https://youtu.be/6zDzuXBCTB8
This appears similar to #1089 #1069 #690
Steps to reproduce:
To reproduce the issue:

Create new Unity project
Import Firebase Unity SDK
Create a method/function which tries to login with Firebase Auth, and a way to call that method/function.
Build a standalone executable
4a. Assuming the build is in a folder that does not contain any special characters, the login attempt should succeed at this point
Rename the directory where the build is located to something which contains a special character
Attempt to login and the following error will be thrown:

InitializationException:  Firebase app creation failed.
  at Firebase.FirebaseApp.CreateAndTrack (Firebase.FirebaseApp+CreateDelegate createDelegate, Firebase.FirebaseApp existingProxy) [0x000da] in Z:\tmp\tmp.uQUresKAQs\firebase\app\client\unity\proxy\FirebaseApp.cs:597
  at Firebase.FirebaseApp.Create () [0x00000] in Z:\tmp\tmp.uQUresKAQs\firebase\app\client\unity\proxy\FirebaseApp.cs:124
  at Firebase.FirebaseApp.get_DefaultInstance () [0x0000b] in Z:\tmp\tmp.uQUresKAQs\firebase\app\client\unity\proxy\FirebaseApp.cs:99
  at Firebase.Auth.FirebaseAuth.get_DefaultInstance () [0x00000] in Z:\tmp\tmp.iEHLvp1aL4\firebase\auth\client\unity\proxy\FirebaseAuth.cs:295
  at TestScript.Update () [0x00053] in C:\Users\work\Desktop\FirebaseMinimalTest\Assets\TestScript.cs:31

(Filename: Z:/tmp/tmp.uQUresKAQs/firebase/app/client/unity/proxy/FirebaseApp.cs Line: 597)

Unable to load Firebase app options ([C:/Users/work/Desktop/FirebaseMinimalTest/Buildé/FirebaseMinimalTest_Data/StreamingAssets\google-services-desktop.json, C:/Users/work/Desktop/FirebaseMinimalTest/Buildé/FirebaseMinimalTest_Data/StreamingAssets\google-services.json] are missing or malformed)
UnityEngine.StackTraceUtility:ExtractStackTrace ()
UnityEngine.DebugLogHandler:LogFormat (UnityEngine.LogType,UnityEngine.Object,string,object[])
UnityEngine.Logger:Log (UnityEngine.LogType,object)
UnityEngine.Debug:LogError (object)
Firebase.Platform.FirebaseLogger:LogMessage (Firebase.Platform.PlatformLogLevel,string) (at Z:/tmp/tmp.cEjnms3lS8/firebase/app/client/unity/src/Unity/FirebaseLogger.cs:95)
Firebase.LogUtil:LogMessage (Firebase.LogLevel,string) (at Z:/tmp/tmp.uQUresKAQs/firebase/app/client/unity/proxy/LogUtil.cs:69)
Firebase.LogUtil:LogMessageFromCallback (Firebase.LogLevel,string) (at Z:/tmp/tmp.uQUresKAQs/firebase/app/client/unity/proxy/LogUtil.cs:77)
Firebase.AppUtil:PollCallbacks () (at Z:/tmp/tmp.uQUresKAQs/firebase/app/client/unity/proxy/AppUtil.cs:32)
Firebase.Platform.FirebaseAppUtils:PollCallbacks () (at Z:/tmp/tmp.uQUresKAQs/firebase/app/client/unity/proxy/FirebaseAppUtils.cs:33)
Firebase.Platform.FirebaseHandler:Update () (at Z:/tmp/tmp.cEjnms3lS8/firebase/app/client/unity/src/Unity/FirebaseHandler.cs:208)
Firebase.Platform.FirebaseMonoBehaviour:Update () (at Z:/tmp/tmp.cEjnms3lS8/firebase/app/client/unity/src/Unity/FirebaseMonoBehaviour.cs:45)

(Filename: Z:/tmp/tmp.uQUresKAQs/firebase/app/client/unity/proxy/LogUtil.cs Line: 69)

Hi @jiahengnz,
Thanks for reporting this issue. Could you provide the sample code you used so that we can identify what's causing this behavior? Could you also confirm if the error message also appears in the Unity editor?

Hi,
The error message does appear in the editor after renaming the project folder to FirebaseMinimalTést.
Source code attached below.
Run in editor/make a build as it is and it will login fine. Change the project name or build folder name to, say, FirebaseMinimalTést, and it will throw the error

Hi @jiahengnz,
I've downloaded the sample project you provided, but I wasn't able to replicate your issue. However, I should point out that I'm using version 2021.2.3f1 of the Unity editor. Could you try using an updated editor and see if the issue persists?

Hi,
Just tested in 2021.2.4f1, the bug occurs in both the editor and the build. Have you renamed the project folder to one with a character like é? The zip as it is will run with no problems.
Unable to load Firebase app options ([C:/Users/work/Desktop/FirebaseMinimalTést/Assets/StreamingAssets\google-services-desktop.json, C:/Users/work/Desktop/FirebaseMinimalTést/Assets/StreamingAssets\google-services.json] are missing or malformed) UnityEngine.Debug:LogError (object) Firebase.Platform.FirebaseLogger:LogMessage (Firebase.Platform.PlatformLogLevel,string) (at Z:/tmp/tmp.cEjnms3lS8/firebase/app/client/unity/src/Unity/FirebaseLogger.cs:95) Firebase.LogUtil:LogMessage (Firebase.LogLevel,string) (at Z:/tmp/tmp.uQUresKAQs/firebase/app/client/unity/proxy/LogUtil.cs:69) Firebase.LogUtil:LogMessageFromCallback (Firebase.LogLevel,string) (at Z:/tmp/tmp.uQUresKAQs/firebase/app/client/unity/proxy/LogUtil.cs:77) Firebase.AppUtil:PollCallbacks () (at Z:/tmp/tmp.uQUresKAQs/firebase/app/client/unity/proxy/AppUtil.cs:32) Firebase.Platform.FirebaseAppUtils:PollCallbacks () (at Z:/tmp/tmp.uQUresKAQs/firebase/app/client/unity/proxy/FirebaseAppUtils.cs:33) Firebase.Platform.FirebaseHandler:Update () (at Z:/tmp/tmp.cEjnms3lS8/firebase/app/client/unity/src/Unity/FirebaseHandler.cs:208) Firebase.Platform.FirebaseMonoBehaviour:Update () (at Z:/tmp/tmp.cEjnms3lS8/firebase/app/client/unity/src/Unity/FirebaseMonoBehaviour.cs:45)

Hi @jiahengnz,
It turns out I didn't encounter this issue at first because I was using a macOS device. I recently tested this on a Windows machine, and I was able to encounter the same error message.
I'll be marking this as a bug for now. Let me discuss this with the team, and I'll get back to you once I have updates.

Excellent, I look forward to hearing back from you


New NuGet Package?

I see there have been a couple recent fixes (for me, I want api_key fix). Is there any chance you could release a new NuGet package with these?

Planned release for 10/29


Fix hosting local item assets

As a temporary workaround, host online.

Could we host in Firebase storage?

There was some issue about this (@LeoLaugier or @TiboX2021 may be able to explain further); you'll see that currently, we host on Github and then point to the raw user content URL 🙃

We could use Firebase storage for static assets, but given the small amount of images we host, I find it overkill. This was an issue with the Angular app, we will have to see how to do it with the lit app (which might not require the rawgithubusercontent workaround). @vivtsai if you want, I can setup a Dockerfile for the lit frontend and test it for url prefixes (at EPFL we deploy the frontend on a generic domain with a url prefix, which is sometimes problematic for app that use relative static asset paths)

Ah, I see. Yes, if you're able to test, that would be great. Presumably if the favicon (the only stored image we have right now) works (or we can get it to work), we'll be able to similarly store the Lost at Sea images?


Openssl Engine under EDKII

Hi，I have a question about openssl engine to ask.
1.Firstly, we can use a crypto accelerator like QAT of Intel and UADK of Huawei under Linux.The commad like this:
openssl dgst -engine uadk_engine test.txt

2.Does the OpenSSL engine support EDKII?
(1) We know that Openssl load the engine share object to connect accelerator driver.
(2) But there is no share object under EDKII.
(3) Is it possible to use accelerator driver under EDKII openssl?

We do not produce builds for EDKII so we unfortunately cannot answer your questions.
But I suppose that no, engines are not supported in UEFI builds and thus accelerators are not supported.

Oh, thank you for your prompt answer. I have another question about openssl engine.
Is it possible to use accelerator driver under Windows like QAT of Intel?
We know that Openssl load the engine share object to connect accelerator driver. But how about dll file under windows.?
Is there a way to load the engine dll file to connect accelerator driver, so that opsnssl can use the accelerator under windows?

Engines are supported on Windows in form of .dll loadable modules. Whether any particular accelerator engine works under Windows is a different question that I do not know the answer for.

Thank you for your answer. I get it.


Performance: Modify bounds detection to inheritance when clipping is disabled

This PR changes the CoreNode update behavior, including how bounds detection works. Instead of creating local strictBoundaries and preLoad boundaries, the boundaries are inherited from the parent unless clipping is enabled.
Why?
Bounds detection added a significant CPU impact to the the rendering loop of the L3 Renderer, causing low end devices to struggle to keep up as the CPU would be overloaded and not in time enough to provide new render instructions to the GPU.
Bounds detection is needed to ensure we only draw what is required on screen / view port and do not render nodes that are outside of the view ports bounds.
What changed?
Previously every node would on every update() calculate what its strictBound and preloadBounds where based off of its world position. This is quite expensive to do and most of time time not needed unless clipping is enabled on a particular node.
This changes:

strictBounds and preloadBounds are inherited from the parent, if the parent has no bounds the viewport stage bounds are used.
If a node enabled clipping, only that parent will calculate it's own strictBound and preloadBound for it's own children
This is calculated once for all children
Don't process anything that's not needed when out of bounds

Unrelated to bounds but performance changes that I ran into:

Only run through clipping if clipping is enabled, previously clipping was executed every global Transform (== expensive)
Minor conditional checks and small performance gains

Test results
Prior to the PR I ran two tests, a CoreNode.update() throughput test (using #364) and a stress benchmark with bounds for FPS measurements.
Tested on a Ryzen 7 6800H / 3070 win11 machine using Chrome Version 128.0.6613.113 (Official Build) (64-bit) on 20x slowdown
Baseline
FPS
index.ts:292 Average FPS: 30.33
index.ts:293 Median FPS: 31
index.ts:294 P01 FPS: 20
index.ts:295 P05 FPS: 25
index.ts:296 P25 FPS: 29
index.ts:297 Std Dev FPS: 2.8001964216818784
index.ts:298 Num samples: 100
index.ts:299 ---------------------------------
Throughput
┌───┬───────────┬─────────┬───────────────────┬────────┬─────────┐
│   │ Task Name │ ops/sec │ Average Time (ns) │ Margin │ Samples │
├───┼───────────┼─────────┼───────────────────┼────────┼─────────┤
│ 0 │ update    │ 126     │ 7899612.499999996 │ ±2.44% │ 64      │
└───┴───────────┴─────────┴───────────────────┴────────┴─────────┘

These changes
FPS
index.ts:292 Average FPS: 40.73
index.ts:293 Median FPS: 41
index.ts:294 P01 FPS: 33
index.ts:295 P05 FPS: 35
index.ts:296 P25 FPS: 39
index.ts:297 Std Dev FPS: 3.3700296734598636
index.ts:298 Num samples: 100
index.ts:299 ---------------------------------
Throughput
┌───┬───────────┬───────────┬───────────────────┬────────┬─────────┐
│   │ Task Name │ ops/sec   │ Average Time (ns) │ Margin │ Samples │
├───┼───────────┼───────────┼───────────────────┼────────┼─────────┤
│ 0 │ update    │ 4,295,328 │<PHONE_NUMBER>198926 │ ±1.28% │ 2147665 │
└───┴───────────┴───────────┴───────────────────┴────────┴─────────┘

About ~10 FPS on 20x slowdown and going from 126 to 4k ops in throughput.

I removed some duplicate code and improved the stage boundary reuse condition. Other than that,  LGTM :)


Change microbatch lookback default from 0 to 1

Resolves #10867
Problem
A lookback value of 0 results in microbatch models consistently having incomplete data.
Solution
Change the lookback default to 1. This guarantees that batches will eventually get completed (assuming the job is run once per batch_size)
Checklist

[X] I have read the contributing guide and understand what's expected of me.
[X] I have run this code in development, and it appears to resolve the stated issue.
[X] This PR includes tests, or tests are not required or relevant for this PR.
[X] This PR has no interface changes (e.g., macros, CLI, logs, JSON artifacts, config files, adapter interface, etc.) or this PR has already received feedback and approval from Product or DX.
[X] This PR includes type annotations for new and modified functions.

Changes to schemas.getdbt.com https://github.com/dbt-labs/schemas.getdbt.com/pull/68

hey @QMalcolm , when this pr is merged, am i right in changing the docs here from 0 to1? https://docs.getdbt.com/docs/build/incremental-microbatch#relevant-configs

hey @QMalcolm , when this pr is merged, am i right in changing the docs here from 0 to1? https://docs.getdbt.com/docs/build/incremental-microbatch#relevant-configs

That is correct 🙂

docs pr here to address this: https://github.com/dbt-labs/docs.getdbt.com/pull/6351/files

Something weird happened. Some commits from another one of my branches got onto this one. I assume in my delirium of spinning plates, I did something dumb.


Main

@Senait-coding you need to refresh your main branch with the latest state of the group's repo and then create a branch for your PR. If you don't do that, you'll keep submitting PRs with commits that are unrelated.

@Senait-coding you need to refresh your main branch with the latest state of the group's repo and then create a branch for your PR. If you don't do that, you'll keep submitting PRs with commits that are unrelated.

yes, may be it is not the right way but last time how I updated my fork with the main(origin)  the steps
I follow were
I make comparison (origin main vs my forked main)
issue PR
merge
got updated fork
clone to my machine
Do the update
push back to my repo
then issue PR to  the origin ; this  way seems ok for me  but it is not working now the moment I issue PR based on the comparison to do the merge it reaches you and I can't merge and update my fork :(, which way should I have to follow? a bit confused

It isn't the good way. Your main must be in the same state as the origin main. By issuing a PR, you are changing the history and that leads to problem like this. Also please don't use the GitHub interface for any of that, you should run those commands on your machine. Daniel just shared some info about that.
In your case you'll have to reset your main something like:
git checkout main
git fetch upstream
git reset upstream/main --hard

Note that doing so you'll lose the work in this PR.

It isn't the good way. Your main must be in the same state as the origin main. By issuing a PR, you are changing the history and that leads to problem like this. Also please don't use the GitHub interface for any of that, you should run those commands on your machine. Daniel just shared some info about that.
In your case you'll have to reset your main something like:
git checkout main
git fetch upstream
git reset upstream/main --hard

Note that doing so you'll lose the work in this PR.

ok, thanks for your explanation and I will try to do it this way, as it worked last time I thought it is the right way to do that

and @Senait-coding it looks like there are actually no new changes that you requested to add..the group repo README.md (which is the project description) is already there


[CoreLocation]  This method can cause UI unresponsiveness if invoked on the main thread

🐛 Bug Report
I am seeing this "warning" in my logs :
[CoreLocation] This method can cause UI unresponsiveness if invoked on the main thread. Instead, consider waiting for the `-locationManagerDidChangeAuthorization:` callback and checking `authorizationStatus` first.

Expected behavior
I would like my app to not experience any "UI unresponsiveness" if it's possible to avoid it.
Reproduction steps
Run the sample app on an iOS 16 device.
Version: 9.0.2
Platform:

[ ] :iphone: iOS

Same issue here!

SAME HERE

SAME HERE. Also experiencing the UI unresponsiveness on IOS 16

@JulianBissekkou
is there any thing we can do to skip this issue ? , we had to lunch an app next week.

My app was rejected yesterday because the screen went unresponsive and while it wasn't possible to reproduce the error, this warning showed up. I need to fix this ASAP to release an important update.

I did a quick fix and the warning is not present anymore.
See: https://github.com/Baseflow/flutter-geolocator/pull/1218
You guys can do some testing and let me know if its works.

the issue still happening

Same here on 29 April 2023.

Hello Everybody, I just solved UI problem with the code below. Still got error but at least UI works with the last known location.
Future<Position?> getUserUpdatedLocationFuture() async {
try {
Position? userPosition = await Geolocator.getCurrentPosition(
desiredAccuracy: LocationAccuracy.best,
timeLimit: const Duration(seconds: 3),
);
return userPosition;
} catch (e) {
return await Geolocator.getLastKnownPosition();
}
}

Same here. Even though I am not running into any UI unresponsiveness, but it is logging in console.
[CoreLocation] This method can cause UI unresponsiveness if invoked on the main thread. Instead, consider waiting for the `-locationManagerDidChangeAuthorization:` callback and checking `authorizationStatus` first.
[CoreLocation] This method can cause UI unresponsiveness if invoked on the main thread. Instead, consider waiting for the `-locationManagerDidChangeAuthorization:` callback and checking `authorizationStatus` first.

@arnabonetraker @angelschnider see my reply , maybe helps #1128 (comment)

:-( unfortunately that's not changing my issue, but that give me an another point of view for searching a fix. Thanks! :-)

SO ANY UPDATES???

@fatihmerickoc, June 5th we released a solution. Shouting in all caps is not the way to get our sympathie.

@mvanbeusekom, thanks for the update to geolocator_apple that fixed this. However I'm "sort of" still being impacted by this. I'm new to coding and flutter so probably missing something obvious. Any help would be greatly appreciated.
My app is dependent on flutter_map_location_marker: ^7.0.2 which is dependent on geolocator: ^9.0.2 which I believe is dependent on geolocator_apple: ^2.1.1+1 (which doesn't have the fix).
I've tried adding geolocator_apple: ^2.2.7 to my dependency_overrides, in the hope this would override the dependency within flutter_map_location_marker. However this doesn't seem to have an effect.
I believe I could create a fork of flutter_map_location_marker: ^7.0.2 to fix this. However I'm just wondering if there is any other solution? Other than waiting for a new version of geolocator to contain geolocator_apple: ^2.2.7 and then wait for an update to flutter_map_location_marker? Many thanks!

Hi @RedHappyLlama,
It should be possible to run the flutter pub upgrade command.
This will update all minor and patch versions of packages that have a dependency with the ^ (caret) constraint. This includes dependencies of dependencies.
More details on package versions and constraints can be found here: https://dart.dev/tools/pub/dependencies#version-constraints
Hope this helps.

@mvanbeusekom, thanks! I've given it a go but still get the issue. I'll raise an issue on flutter_map_location_marker to see if it's an issue with their package.


"Bad return status: 400" for  SM-S908B region EUX

trying to download SM-S908B with EUX results in

Bad return status: 400

SM-S908B with EUX definitely exist, here is a screenshot as proof:

and here is a screenshot of trying to download it with Bifrost:

interestingly, same happens with EUY but /not/ with BTE (BTE works fine).

same

@Xpl0itR until this is resolved, i wrote a region scanner to find alternative regions at https://github.com/jesec/SamFirm.NET/issues/3
$ time php test2.php SM-F916N
total urls to check per model string: 509
total urls to check: 2545
Checking model string 1/5: SM-F916N
http://fota-cloud-dn.ospserver.net/firmware/KTC/SM-F916N/version.xml   : 200
http://fota-cloud-dn.ospserver.net/firmware/LUC/SM-F916N/version.xml   : 200
http://fota-cloud-dn.ospserver.net/firmware/SKC/SM-F916N/version.xml   : 200
http://fota-cloud-dn.ospserver.net/firmware/KOO/SM-F916N/version.xml   : 200
(...)

in my case I couldn't download the correct EUX-region firmware, but my scanner found a compatible region BTE which worked fine

@MichaelDavidGK because my phone's model code is SM-S908BZWGEUB , but the update server did not use my full model code, and I wasn't sure exactly what model code the update servers used, so after an exhaustive search,
hans@cac3:~$ time php test2.php SM-S908BZWGEUB
total urls to check per model string: 509
total urls to check: 5599
Checking model string 1/11: SM-S908BZWGEUB
Checking model string 2/11: SM-S908BZWGEU
Checking model string 3/11: SM-S908BZWGE
Checking model string 4/11: SM-S908BZWG
Checking model string 5/11: SM-S908BZW
Checking model string 6/11: SM-S908BZ
Checking model string 7/11: SM-S908B
http://fota-cloud-dn.ospserver.net/firmware/AFG/SM-S908B/version.xml   : 200
http://fota-cloud-dn.ospserver.net/firmware/MOB/SM-S908B/version.xml   : 200
http://fota-cloud-dn.ospserver.net/firmware/VIP/SM-S908B/version.xml   : 200
http://fota-cloud-dn.ospserver.net/firmware/O2C/SM-S908B/version.xml   : 200
(...)

it turns out that the update server used the model code SM-S908B even though the Samsung warranty page use the model code SM-S908BZWGEUB

also even if it exists it's a different model, so there is no point to it

I certainly hope not (and the SM-S908B firmware was fully compatible with my SM-S908BZWGEUB model.)

There isn't really much I can do if Samsung isn't serving the firmware for a certain CSC directly. As suggested above, you'll have to try a different CSC that contains yours.

@zacharee problem is, the Samsung phone itself can download  SM-S908B region EUX, but not Bifrost - The phone is doing /something/ differently, and the server is reacting to it

OTA packages use a different server and require a pretty complex registration process that's unique per-device.

@zacharee oh ok. close this issue then?
Btw would be nice if Bifrost itself could get a region scanner (I don't do Kotlin myself, but I wrote one in PHP to get around the  SM-S908B+EUX issue - would have saved me the trouble if Bifrost had one)

Can you request that in a new issue so I can track it better?

I experience the same problem with Bifrost 1.19.3 when trying to download S928BXXS3AXI1/S928BOXM3AXI1/S928BXXS3AXI1/S928BXXS3AXI1 for SM-S928B and EUX CSC. The update definitely exists, I got an OTA notification for it and it is present in samfw https://samfw.com/firmware/SM-S928B/EUX/S928BXXS3AXI1


ERROR:  Exception thrown while making HTTP request!

Hi,
I have configured OSF plugin to development environment as below, it worked all good but when I changed the host-name to my staging environment(staging-.demandware.net) it started failing with the error below. Can someone please help me in understanding what I am doing wrong.
Thank you
Plugin configuration in jenkins file -
osfBuilderSuiteForSFCCDeploy(
hostname: "",
bmCredentialsId: "abcdefghijklmn",
ocCredentialsId: "abcdefghijklmnopqr",
ocVersion: 'v19_5',
buildVersion: "dev",
createBuildInfoCartridge: true,
sourcePaths: [
[sourcePath: "src/cartridges"]
],
actuvateBuild: true,
tempDirectory: 'tmp/build'
)
error-
ERROR:
Exception thrown while making HTTP request!
org.apache.http.NoHttpResponseException: staging-.demandware.net:443 failed to respond
at org.apache.http.impl.conn.DefaultHttpResponseParser.parseHead(DefaultHttpResponseParser.java:141)
at org.apache.http.impl.conn.DefaultHttpResponseParser.parseHead(DefaultHttpResponseParser.java:56)
at org.apache.http.impl.io.AbstractMessageParser.parse(AbstractMessageParser.java:259)
at org.apache.http.impl.DefaultBHttpClientConnection.receiveResponseHeader(DefaultBHttpClientConnection.java:163)
at org.apache.http.impl.conn.CPoolProxy.receiveResponseHeader(CPoolProxy.java:157)
at org.apache.http.protocol.HttpRequestExecutor.doReceiveResponse(HttpRequestExecutor.java:273)
at org.apache.http.protocol.HttpRequestExecutor.execute(HttpRequestExecutor.java:125)
at org.apache.http.impl.execchain.MainClientExec.execute(MainClientExec.java:272)
at org.apache.http.impl.execchain.ProtocolExec.execute(ProtocolExec.java:185)
at org.apache.http.impl.execchain.RetryExec.execute(RetryExec.java:89)
at org.apache.http.impl.execchain.RedirectExec.execute(RedirectExec.java:110)
at org.apache.http.impl.client.InternalHttpClient.doExecute(InternalHttpClient.java:185)
at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:83)
at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:108)
at org.jenkinsci.plugins.osfbuildersuiteforsfcc.deploy.OpenCommerceAPI.createCodeVersion(OpenCommerceAPI.java:173)
at org.jenkinsci.plugins.osfbuildersuiteforsfcc.deploy.DeployBuilder$DeployCallable.invoke(DeployBuilder.java:1371)
at org.jenkinsci.plugins.osfbuildersuiteforsfcc.deploy.DeployBuilder$DeployCallable.invoke(DeployBuilder.java:534)
at hudson.FilePath$FileCallableWrapper.call(FilePath.java:3042)
at hudson.remoting.UserRequest.perform(UserRequest.java:212)
at hudson.remoting.UserRequest.perform(UserRequest.java:54)
at hudson.remoting.Request$2.run(Request.java:369)
at hudson.remoting.InterceptingExecutorService$1.call(InterceptingExecutorService.java:72)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:748)

Based on the error org.apache.http.NoHttpResponseException the SFCC server did not returned a valid HTTP response. I assume you used the correct hostname and not staging-.demandware.net.
Are you getting this error every time you run the build? Or was it one time only? Can you make a HTTPS request to same hostname using something else like curl or Postman?

I am getting this error every time I give staging hostname  (staging-realm-customer.demandware.net) if I replace hostname with development (development-realm-customer.demandware.net) its working fine.
Staging environment:
I ran curl command
H:>curl https://staging-xxxx-xxxxxxx.demandware.net

404 URL Not Found

URL Not Found

                The requested URL was not found on this server.

when I open this url in my browser

Is there any way to send me a private message/email with the actual domain? So I can try and test on my end? I'm not sure how else I can help since I'm not able to reproduce the error on my end.

Actually from what I see the error happens when it attempts to create the code version createCodeVersion and I see you attempt to do this on a staging instance where two factor auth is required to be able to upload code. So you need too setup the two factor auth credentials (client SSL cert/key) and use cert.staging.realm.customer.demandware.net as hostname instead of staging-realm-customer.demandware.net

thank you, we couldnot even push cartridges from local machine, so we believe something is wrong from our end. Will fix it and update you.
Thank you again


Flutter example doesn't work?

It seems the example here: https://github.com/Cretezy/redux_persist/blob/master/packages/redux_persist_flutter/example/redux_persist_flutter_example.dart
doesn't work. First, it requires the WidgetsFlutterBinding.ensureInitialized(); call because otherwise it fails with:
Unhandled Exception: StorageException: On load: ServicesBinding.defaultBinaryMessenger was accessed before the binding was initialized.
If you're running an application and need to access the binary messenger before `runApp()` has been called (for example, during plugin initialization), then you need to explicitly call the `WidgetsFlutterBinding.ensureInitialized()` first.
If you're running a test, you can call the `TestWidgetsFlutterBinding.ensureInitialized()` as the first line in your test's `main()` method to initialize the binding.

When the line is added, it fails with:
Unhandled Exception: SerializationException: On load: NoSuchMethodError: The method '[]' was called on null.
Receiver: null
Tried calling: []("counter")

That's because this function: https://github.com/Cretezy/redux_persist/blob/master/packages/redux_persist_flutter/example/redux_persist_flutter_example.dart#L38
doesn't handle null, which is the initial value the first time the application is ever launched.

had same issue

It seems the example here: https://github.com/Cretezy/redux_persist/blob/master/packages/redux_persist_flutter/example/redux_persist_flutter_example.dart
doesn't work. First, it requires the WidgetsFlutterBinding.ensureInitialized(); call because otherwise it fails with:
Unhandled Exception: StorageException: On load: ServicesBinding.defaultBinaryMessenger was accessed before the binding was initialized.
If you're running an application and need to access the binary messenger before `runApp()` has been called (for example, during plugin initialization), then you need to explicitly call the `WidgetsFlutterBinding.ensureInitialized()` first.
If you're running a test, you can call the `TestWidgetsFlutterBinding.ensureInitialized()` as the first line in your test's `main()` method to initialize the binding.

When the line is added, it fails with:
Unhandled Exception: SerializationException: On load: NoSuchMethodError: The method '[]' was called on null.
Receiver: null
Tried calling: []("counter")

That's because this function: https://github.com/Cretezy/redux_persist/blob/master/packages/redux_persist_flutter/example/redux_persist_flutter_example.dart#L38
doesn't handle null, which is the initial value the first time the application is ever launched.
Same issue. Have you found solution?

@DmitriyGribennikov yes, just add WidgetsFlutterBinding.ensureInitialized(); somewhere before your code using this library if you don't have it yet for issue #1. Add a null check for issue #2.
But I've migrated away from this library


Select first match in quick outline

This has been annoying me for a while.. Quick outline is now:

case insensitive
a matching definition is automatically selected, so you can just press Enter and Go!

The eclipse-mars profile will fail, that's normal, but the Luna one is fine

LGTM


Unversioned Ballerina target directory is created after a successful build

Description:
Recently I noticed that, after a module build, there is an additional directory created called ballerinaTarget. That directory contains an unversioned bala file and this unversioned file will fail our releases. This issue happens after the recent changes to the Gradle plugin (introduction of the target dir) [1]. The possible solutions to fix this:

Ignore the bala files in gitignore
Ignore the entire ballerinaTarget directory in gitignore
Create the ballerinaTarget inside the build directory (since build directory is already ignored, ballerinaTarget  will be automatically ignored)

[1] https://github.com/ballerina-platform/plugin-gradle/pull/65
@niveathika @ThisaruGuruge @shafreenAnfar @daneshk WDYT?.

+1 for the third option. I already cleaned up the paths but did not send a PR since it has many other changes.
https://github.com/ballerina-platform/plugin-gradle/pull/68

Shall we merge this before the release since this will fail our releases?

+1 for the third option. I already cleaned up the paths but did not send a PR since it has many other changes.
ballerina-platform/plugin-gradle#68

Yeah, +1 for the third option.

To fix this issue, the following modules' plugin version needs to be updated to 0.14.1.
Please refer this PR to get more info: https://github.com/ballerina-platform/plugin-gradle/pull/68

[ ] io
[ ] url
[ ] java.arrays
[ ] random
[ ] regex
[ ] task
[ ] time
[ ] xmldata
[ ] crypto
[ ] os
[ ] xslt
[ ] uuid
[ ] log
[ ] cache
[ ] file
[ ] ftp
[ ] mime
[ ] nats
[ ] stan
[ ] tcp
[ ] udp
[ ] auth
[ ] email
[ ] http
[ ] jwt
[ ] oauth2
[ ] graphql
[ ] grpc
[ ] websocket
[ ] websub
[ ] websubhub
[ ] kafka
[ ] rabbitmq
[ ] sql
[ ] java.jdbc
[ ] mysql
[ ] mssql
[ ] oracledb
[ ] postgresql


Refactor/packages

Refactoring package creation scripts for Ubuntu, Debian, CentOS and OS X distributions.

Coverage remained the same at 66.49% when pulling 9fd49e6c46bbb960ed8a4595e03bb85a7ce34e3f on refactor/packages into 2ce9eeb6965623d150c33d9fef44b7b7aa5e7623 on master.

Coverage remained the same at 66.49% when pulling 9fd49e6c46bbb960ed8a4595e03bb85a7ce34e3f on refactor/packages into 2ce9eeb6965623d150c33d9fef44b7b7aa5e7623 on master.

Coverage remained the same at 66.49% when pulling 9fd49e6c46bbb960ed8a4595e03bb85a7ce34e3f on refactor/packages into 2ce9eeb6965623d150c33d9fef44b7b7aa5e7623 on master.


Auto-complete popup puts redundant "item" prefix per suggested code for screen readers

CC @isidorn
This issue has been also filed on NVDA repo #11404, and confirmed that the fix has to be addressed on VSCode end.
System Info
Version: 1.48.0-insider (system setup)
Commit: 0a7364f00514c46c9caceece15e1f82f82e3712f
Date: 2020-07-22T05:49:27.463Z
Electron: 9.1.0
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Microsoft® Windows® Operating System Version 10.0.19041.1
NVDA v2020.1
Steps to Reproduce:

With NVDA loaded, create any source code file in VSCode (e.g., test.py).
Try using code suggestion feature (e.g., type cl and press Ctrl+Space and arrow-down to find class keyword).

Actual behavior:
NVDA and other screen readers (e.g., JAWS) keep announcing redundant prefix, "item," per suggested code in the auto-complete popup.
Expected behavior:
It would be ideal to filter out this prefix for NVDA users to quickly navigate through suggested code items.
Does this issue occur when all extensions are disabled?: Yes

@jooyoungseo thanks for feedback. I have pushed a fix for this. You can try it out with vscode insiders from Wednesday and let us know if it is good for you.

Thanks very much @isidorn! I will try that out on Wednesday, and keep you posted.

I have confirmed that this issue has been resolved via the latest commit:
Version: 1.48.0-insider (system setup)
Commit: 15e798e081984a17eae00de91adf4479190d5ed2
Date: 2020-07-28T10:39:18.007Z
Electron: 9.1.0
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19041
Thanks very much again @isidorn for this fix!

Great, thanks for letting us know. Adding verified label.


Repeated double-tap should zoom out in photo view

Current behaviour:
Repeated double tap on photo continues to increase zoom
Proposed behaviour:
Second double tap should revert zoom back to original.

That big zoom value makes no sense anyway
It will allow to zoom-out with single hand (currently impossible)

Should be fixed on prod now 🙇♂

@cheeaun i really don't like 5 levels of zoom :(
zoom-in/zoom-out taps should be enough in my opinion. Other zoom levels can be reached with 2 fingers swipe. It's too hard to exit from zoom with so many taps.
Can this be reconsidered?


Change column 'Username' in table 'Authorities' to varchar(64) #1467

Change column 'Username' in table 'Authorities' to varchar(64) #1467

Codecov Report

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

@@             Coverage Diff              @@
##             master    #1471      +/-   ##
============================================
+ Coverage      48.4%   48.46%   +0.05%
  Complexity     1801     1801
============================================
  Files           379      379
  Lines         10930    10930
  Branches       1121     1121
============================================
+ Hits           5291     5297       +6
+ Misses         5253     5244       -9
- Partials        386      389       +3

Impacted Files
Coverage Δ
Complexity Δ

...ervice/service/ReleaseMessageServiceWithCache.java
85.71% <0%> (-1.2%)
24% <0%> (-1%)

...lo/configservice/util/InstanceConfigAuditUtil.java
70.17% <0%> (-0.88%)
13% <0%> (-1%)

...work/apollo/biz/message/DatabaseMessageSender.java
66.66% <0%> (+16.66%)
8% <0%> (+2%)
:arrow_up:

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 976b69d...e2fc3e9. Read the comment docs.

Coverage increased (+0.08%) to 52.022% when pulling e2fc3e9172119ce336a9622b9573ae375ed59477 on ryphnyuan:master into 976b69d019e6c8be409ed93c7a1dc861bbc82643 on ctripcorp:master.

@ryphnyuan
Thanks!
Would you please help to fix the other 2 sqls?
https://github.com/ctripcorp/apollo/blob/master/scripts/docker-quick-start/sql/apolloportaldb.sql
https://github.com/ctripcorp/apollo/blob/master/scripts/apollo-on-kubernetes/db/portal-db/apolloportaldb.sql

Great!
Normally, we would squash commits for one feature into one commit. Would you please help to do so?
There are 2 ways to do this:

To rebase and squash based on the remote branch

git rebase -i /master

Create a new branch and merge these commits into one

git checkout -b  /master
git merge --squash master

I shall start a new pull request.