(Fix) - Fixs tab selection

Closes #956

Travis automatic deployment:
https://pr960--safereact.review.gnosisdev.com/app

So the link works, but I have other issues that only happen in this PR (i've checked with dev to compare)
Loading a this safe 0x9913B9180C20C6b0F21B6480c84422F6ebc4B808 for the first time shows this error

It happens every time I clear chache and storage and load the safe again
Another Issue I had in that safe is that the collectibles were not loading. The "Manage List" modal shows an infinite loading icon

Those errors aren't related to this pr, specially the first one, that was an error on dev, now it's fixed on dev. I'll sync this branch with dev also

Ok, if these issues have nothing to do with the PR I'll approve it

  useEffect(() => {
    const showCollectibles = COLLECTIBLES_LOCATION_REGEX.test(currentPath)
    const showCoins = COINS_LOCATION_REGEX.test(currentPath)
    const subMenuOptions = [{ enabled: showCoins, legend: 'Coins', url: `${SAFELIST_ADDRESS}/${address}/balances` }]

    if (!showCollectibles && !showCoins) {
      history.replace(`${SAFELIST_ADDRESS}/${address}/balances`)
    }

    const erc721Enabled = featuresEnabled && featuresEnabled.includes('ERC721')

    if (erc721Enabled) {
      subMenuOptions.push({
        enabled: showCollectibles,
        legend: 'Collectibles',
        url: `${SAFELIST_ADDRESS}/${address}/balances/collectibles`,
      })
    } else {
      if (showCollectibles) {
        history.replace(subMenuOptions[0].url)
      }
    }

  }, [featuresEnabled, address, currentPath])

Look at showCollectibles and showCoins, we use it in order to update the styling of the current selected tab

@Agupane this would be more easy with react-router since NavLink component has activeClassName prop - https://reacttraining.com/react-router/web/api/NavLink

@Agupane this would be more easy with react-router since NavLink component has activeClassName prop - https://reacttraining.com/react-router/web/api/NavLink

I refactored it, please give it a look @mikheevm

Travis automatic deployment:
https://pr960--safereact.review.gnosisdev.com/app


-Whatif parameter doesn't work on New/Set/Remove-ApplicationAccessPolicy.md

https://github.com/MicrosoftDocs/office-docs-powershell/issues/5146

@chrisda Do you think we must report it? Thanks!

Docs Build status updates of commit 5292ba9:
:clock10: Full build: the average full build time is 12 min(s) 29 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 5292ba9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/organization/New-ApplicationAccessPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/organization/Remove-ApplicationAccessPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/organization/Set-ApplicationAccessPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit dcb5ab4:
:clock10: Full build: the average full build time is 12 min(s) 29 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit dcb5ab4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/organization/New-ApplicationAccessPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/organization/Remove-ApplicationAccessPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/organization/Set-ApplicationAccessPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit d24279f:
:clock10: Full build: the average full build time is 12 min(s) 29 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit d24279f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/organization/New-ApplicationAccessPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/organization/Remove-ApplicationAccessPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/organization/Set-ApplicationAccessPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix #319 Failing to parse some PackageJson Person fields

Fix issues while parsing Person package.json fields

we were not properly handling cases when the name, email, and url fields
were out of order.
fix assertion check when result is empty
add relevant test cases

:+1:


type refactor

Can we fix these 3 build errors?
ERROR in /home/runner/work/vscode-marquee/vscode-marquee/packages/extension/tests/extension.test.ts
67:44-47
[tsl] ERROR in /home/runner/work/vscode-marquee/vscode-marquee/packages/extension/tests/extension.test.ts(67,45)
      TS2322: Type 'string' is not assignable to type 'never'.

ERROR in /home/runner/work/vscode-marquee/vscode-marquee/packages/extension/tests/gui.view.test.ts
56:39-47
[tsl] ERROR in /home/runner/work/vscode-marquee/vscode-marquee/packages/extension/tests/gui.view.test.ts(56,40)
      TS2345: Argument of type 'string' is not assignable to parameter of type 'never'.

ERROR in /home/runner/work/vscode-marquee/vscode-marquee/packages/extension/tests/gui.view.test.ts
59:39-47
[tsl] ERROR in /home/runner/work/vscode-marquee/vscode-marquee/packages/extension/tests/gui.view.test.ts(59,40)
      TS2345: Argument of type 'string' is not assignable to parameter of type 'never'.


Enhancing Metadata on the server.

Hi i figured out how to add additional properties to the model on the server, however, I am a bit challenged client-side as breeze-client complains that it cannot create an instance of DataProperty as it cannot parse my custom properties. Is there a way to have the system use a custom class?
Thanks in advance

I'm not sure what you are trying to do. Can you give a complete example of how you would expect it to work?


Translate 'Getting Started' page to Polish

Getting Started page translated to Polish.

Should be ready to merge now.

Waiting for the other maintainers to approve. We need at least 2 approvals before merging. We don't want biased opinions. What's more, others can spot issues I didn't see and/or discuss about particular ways of translating things, which will be later added to the glossary.

@TeoTN @bartlomiejzuber @marcinostrouch Macie jeszcze jakieś uwagi? Jeśli nie, to dajcie approve, bo się kolejka robi...


feat(Analysis.Seminorm, Analysis.LocallyConvex.WithSeminorms): minimize some assumptions

To be honest most of them don't matter in practice, but I couldn't hold myself :sweat_smile:

bors r+


Passwords are cross-checked against "Have I Been Pwned".

@bullet-train-co please provide an option to disable this.


Query in the Tindroid Application

How to change the color of the right bubble?

https://github.com/tinode/tindroid/blob/master/app/src/main/java/co/tinode/tindroid/UiUtils.java#L107

From where we should change the color?

The color is assigned here:
https://github.com/tinode/tindroid/blob/master/app/src/main/java/co/tinode/tindroid/MessagesListAdapter.java#L345
So, you need to assign bgColor in these two cases:
https://github.com/tinode/tindroid/blob/master/app/src/main/java/co/tinode/tindroid/MessagesListAdapter.java#L326
https://github.com/tinode/tindroid/blob/master/app/src/main/java/co/tinode/tindroid/MessagesListAdapter.java#L337

BTW, I'll move the colors to the resource in the next release.

Thank You.


Styling updates for 2021 dbt rebrand

Description & motivation

Pre-release docs
Is this change related to an unreleased version of dbt?

[x] No: please ensure the base branch is current

I noticed on mobile that the website does not fill the screen on the subpages. I believe this is a viewport issue. The Docusaurus viewport meta tag is set to <meta name="viewport" content="width=device-width" />, whereas the content attribute should also have an initial scale set to 1.
However, even after adding an updated viewport meta tag to the Head component, it looks like Docusaurus overrides that with it's own viewport tag. The only other option I see is to set an ssrTemplate, but that looks like it will reconfigure the entire head based on what is added there.

I see what you mean about the viewport! It happens in production, too:
https://docs.getdbt.com/docs/introduction

(Also you might also want to tag in @malbracht as a reviewer, since this is her design).

Looks like button color still needs to change per comp.
Browser metadata
Path:      /
Browser:   Chrome 92.0.4515.159 on Mac OS 10.15.7
Viewport:  1891 x 857 @1x
Language:  en-US
Cookies:   Enabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved

Actually, never mind, I goofed. It seems like she wants different tiers of buttons. I get it now. Ignore me!

I see what you mean about the viewport! It happens in production, too:
https://docs.getdbt.com/docs/introduction
I assume you tried adding the meta tag to the /website/src/pages/index.js (for example, where the verification meta is?) That's the one that gets overridden?
(Have not reviewed yet; will review now)

Yes you are correct. For example I added in the link tags within that same Head component in the index file, and those are applied. However, if I add the viewport meta tag, it does not appear to update the existing one. I will add the updated meta tag to the latest commit to see if it overrides it on deployment.

lgtm! :shipit:
Jason, can you go ahead and make a new task in Asana to fix the viewport issue? I think you'll need to seek help from Drew on that... he may be the only one who understands how that works.

Will do, creating the task now


Ash Walker`s spear and uniform

About the PR

This PR adds bones as a new material (intended to be drop in the future when there will be more fauna), spear made of bones, both bone helmet and armor and a gladiator`s uniform and helmet.
Why / Balance

I believe this PR will add more possibilities to the future events and other PRs.

for upcoming Lavaland station  #14226

Technical details

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl: Tunguso4ka

add: Added gladiator`s helmet and uniform for both the station and planet gladiators.
add: Added bone armor and spear to please our lava planet habitats.

and yes it works.

worst day of my life.

Why is it trying to find spear1 there?

pro linter broke

sorry... Why is it trying to find spear1 there?

it's unrelated to your changes. Don't worry about it.

it's unrelated to your changes. Don't worry about it.

yay, okay :3


Notebook 2 env file

Notebook 2 should mention the need to update the .env file with cluster id & collection name before the generation of training data. Then with the custom ranker id before the experiment step. Also need to restart the flask server after changing the .env file.

@jrtorres : This is incorporated last week. Right before running experiments I have added a comment to update the .env file.  Thanks for the valuable feedback.


[SPARK-13427][SQL] Support USING clause in JOIN.

What changes were proposed in this pull request?
Support queries that JOIN tables with USING clause.
SELECT * from table1 JOIN table2 USING <column_list>
USING clause can be used as a means to simplify the join condition
when :

Equijoin semantics is desired and
The column names in the equijoin have the same name.

We already have the support for Natural Join in Spark. This PR makes
use of the already existing infrastructure for natural join to
form the join condition and also the projection list.
How was the this patch tested?
Have added unit tests in SQLQuerySuite, CatalystQlSuite, ResolveNaturalJoinSuite

Can one of the admins verify this patch?

@rxin @adrian-wang Can you please review the implementation and let me know your comments.Thanks !!

@rxin Hi Reynold, would appreciate your feedback on this PR. Thanks a lot.

@dilipbiswal Could you please resolve the conflicts? Thanks!

@gatorsmile Thanks !! I have rebased the code now.

We should also modify the resolved attribute of a join to return false until UsingJoin is resolved.

This looks pretty good.  Thanks for working on it!  Can you also update the the function in Dataset to use this code path instead of doing resolution there.

Oh, you already did :)  Ignore that.

ok to test

Test build #53145 has started for PR 11297 at commit f7bb37e.

Test build #53145 has finished for PR 11297 at commit f7bb37e.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Test build #53154 has started for PR 11297 at commit f39f547.

Test build #53154 has finished for PR 11297 at commit f39f547.

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

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

Merged build finished. Test PASSed.

cc @marmbrus

Test build #53397 has started for PR 11297 at commit 29a4f59.

Test build #53397 has finished for PR 11297 at commit 29a4f59.

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

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

Merged build finished. Test PASSed.

LGTM.  Thanks!
Merging to master.

@marmbrus Thank you very much Michael !!


Notify user of supported backend storage

Signed-off-by: Chun Chen<EMAIL_ADDRESS>When I tried to start docker with docker -d  --bip <IP_ADDRESS>/16 -D -H tcp://<IP_ADDRESS>:2375 -H unix:///var/run/docker.sock --kv-store=Zookeeper:<IP_ADDRESS>:2181, I got the error "Backend storage not supported yet, please choose another one". There isn't any docs in docker/libnetwork/libkv telling it's zk instead of Zookeeper. So I think it's better to make the error message self-explaining.

not a maintainer here, but I like this from a user perspective; thanks!

Thanks again. @thaJeztah

LGTM, thanks @chenchun! And thanks for kicking in @thaJeztah :bowtie:


Possible curated link: Documentation, testing, coverage, and publishing of Julia packages

I am wondering whether you'd accept a curated link to the following blog post:
http://www.stochasticlifestyle.com/finalizing-julia-package-documentation-testing-coverage-publishing/
It's on finishing up the packaging toolchain for Julia, a popular scientific computing programming language. It walks though setting up unit tests, continuous integration, coverage, automatic generation of documentation, and publishing to the package repository.

We welcome a submission on this topic.  The current BSSw leadership team does not have an expert in Julia, but we recognize its importance and growing use.
If you want to submit a draft at some point, for more detailed review, please feel free to submit a pull request.
Thank you.

Julia's v1.0 with Pkg3 will be coming out soon. Let me write a new post for that and hold out for it to release.

Great. Also noticed this posting from NVIDIA
https://devblogs.nvidia.com/parallelforall/gpu-computing-julia-programming-language/

The link doesn't work for me.

@jarrah42 same here.  Looks like he may have "rebooted" his blog since the idea was submitted.  We could contact him to see if there is a current alternative.

@jarrah42 same here. Looks like he may have "rebooted" his blog since the idea was submitted. We could contact him to see if there is a current alternative.

Is anyone following up with him? If not, we should move this as "no interest"? or something else since we seem interested in the topic if not the specific article...

The link seems to work for me now.

I down-voted because I would like to avoid articles that focus on a specifc package/product.

So this issue seems to have gathered a second link from Mike. I looked at both and have very different opinions about them.
I did have some difficulty accessing the blog even with the new link. The original request seems to low-level for this venue focusing as it does on specific packaging needs in Julia, package definitions, licensing requirements, etc. I agree with Mark that this is Julia-only content and have down-voted it.
The second link seems more promising as it focuses more on how Julia can be used in a GPU environment. I am going to read this one again and if I think it is of interest I will open a separate issue to consider it.
Paul

So is there an EB policy that articles have to be "generic" in some way? Seems like this is more a failing of the site not providing scope for specific content.

I don't think we have an official policy.  I think some individuals on the EB have opinions.  A big part of why we have these discussions about topic proposals is to achieve consensus on what we think the site should be.  To develop the policy in other words, whether we write it down or not.

So, in re-reviewing...I favor as a CC resource

Is the version of Julia being referenced in article still valid?

PR #443 was created for this topic.This was authored by Marc Miller.


Fixed broken image link for EmbeddedAppExample.png

Image link for EmbeddedAppExample.png failed to load. Fixed link to load original image. Did not touch source code.

I can see the image just fine?
http://facebook.github.io/react-native/docs/embeded-app.html#compile-and-run

I was referring to the page on GitHub directly.
https://github.com/facebook/react-native/blob/master/docs/EmbeddedApp.md

@vjeux This change fixed the image problem on GitHub, but have inadvertently broke the one on the official page. Sent and email to your<EMAIL_ADDRESS>regarding this.


a JavaScript error occurred in the main procees ,is about the electron's error ?

details see the follow pic:

my pc system is win7;
and the software is :云存储管理客户端-0.3.1-win

i just choose some pic to download when the error occurred

把下载目录下的diffLog.log删了好像就可以了


fix patches for databroker 1.2.3

inout in baseline reports values that are labeled as floats, not strings.  The type() should be changed to string
fccd_image shape is taken from top level AD setup.  WRitten files should get image shape from the plugin responsibble for writing the image.

For item 2, the HDF plugin writes the file, and the PVs associated with array shape are :
XF:23ID1-ES{FCCD}HDF1:ArraySize0_RBV
XF:23ID1-ES{FCCD}HDF1:ArraySize1_RBV

The raw images is rotated in CSXtools so the user sees something different.  The unrotated images has the first number being horizontal, and the second number being veritcal.
After processing by CSXtools, image shape flips axes
This is extra confusing because the raw image is not the image handled through the ROI and STATS plugins so don't change sutff associated with ROI and STATS as this represents the postCSXtools process orientation that use users sees activley in imageJ at the beamline.

Item 1 is now being patched at data-access time by a "databroker transform":
# /nsls2/software/etc/tiled/profiles/csx_transforms.py
import copy

def patch_descriptor(doc):
    doc = copy.deepcopy(doc)
    # This was labeled "integer" but it is actually "string".
    INOUT_KEY = "inout_status"
    if INOUT_KEY in doc["data_keys"]:
        doc["data_keys"][INOUT_KEY]["dtype"] = "string"
    return doc

Fixing that at the source wouldn't hurt, but it's low priority.
Item 2 cannot be patched in this way, so it's currently being fixed in databroker migration that is manually run. Fixing this at the source would remove the need for the migration and enable data from CSX to be immediately readable using "secure-able databroker" (i.e. from jupyter.nsls2.bnl.gov).

Notes from fixing Item 2 at the source
Before any changes:
In [20]: RE(count([fccd]))
<snipped>

In [21]: db[-1].descriptors[0]['data_keys']['fccd_image']['shape']
Out[21]: [1, 2050, 1152]

In [22]: np.asarray(list(db[-1].data('fccd_image'))).shape
Out[22]: (1, 1, 1000, 960)

In [23]: fccd.make_data_key()
Out[23]:
{'shape': (1, 2050, 1152),
 'source': 'PV:XF:23ID1-ES{FCCD}',
 'dtype': 'array',
 'external': 'FILESTORE:'}

After:
In [1]: fccd.make_data_key()
Out[1]:
{'shape': (1, 1000, 960),
 'source': 'PV:XF:23ID1-ES{FCCD}',
 'dtype': 'array',
 'external': 'FILESTORE:'}

In [6]: db[-1].descriptors[0]['data_keys']['fccd_image']['shape']
Out[6]: [1, 1000, 960]

In [5]: RE(count([fccd]))
<snipped>

In [7]: np.asarray(list(db[-1].data('fccd_image'))).shape
Out[7]: (1, 1, 1000, 960)

The fix is https://github.com/NSLS-II-CSX/xf23id1_profiles/commit/7d74ef0c7f78e01ada597713b652fd7a20a743ec.

Having fixed the way shape metadata is written, I then hand-edited /etc/databroker/csx.yml to point to the migrated11 database so that the bsui process will write to that database directly. This change will need to be applied on all hosts where bsui is run and captured in configuration control.
I also went back for the old data that we had not yet migrated (30 May to the present). All data back to 2017 should now be readable.

The change was manually applied to:
  614  ssh xf23id1-ws1.nsls2.bnl.local
  615  ssh xf23id1-srv1.nsls2.bnl.local
  616  ssh xf23id1-ws4.nsls2.bnl.local
  617  ssh xf23id1-ws5.nsls2.bnl.local
  618  ssh xf23id1-ws3.nsls2.bnl.local
  619  ssh xf23id1-srv2.nsls2.bnl.local
  620  ssh xf23id1-srv3.nsls2.bnl.local
  621  ssh xf23id1-srv3.nsls2.bnl.local

because Satellite was not working, but the change has been merged into the devops repo.


Add story to demonstrate ref forwarding in Vulcan Components

This story demonstrates a solution to https://github.com/VulcanJS/Vulcan/issues/2449.

Thanks!


Footermod

Summary of changes
Modify Custom Footer to add the social media icons for ovc footers
check to show that non ovc footers are not effected
Frontend

Add React Icons to packages
Add footer-social.js (to blocks/ovc)

custom code to add social media icons

Modified customFooter.js to check if unit tag is OVC and if so - add social icons below the image.
Add social-icons.css - to remove some styles that were interfering with how icons looked

Backend

None

Test Plan
https://ovccontenthub.gatsbyjs.io/ovc/gceb3 icons at the bottom should work as on https://ovc.uoguelph.ca/
https://ovccontenthub.gatsbyjs.io/test-media-text-in-section - a test page to show that if not an ovc footer - icons don't display
https://ovccontenthub.gatsbyjs.io/cancer-to-delete (or any page that does not have a footer) no change or footer elements added

Abandoned in favour of the one modified by Miranda.


github action fails on branch

this action fails, but not for the reason I thought it would!
https://github.com/ultrasaurus/webgenr/actions/runs/4653065984

fixed


Exclude Runtime Assembly From Game Builds

Hi, the usability of this package is awesome!
One minor issue I have found is with your runtime classes: they are included in game builds by default.
Just set Auto Referenced to false in the runtime asmdef.

Also setting overrideReferences to true in asmdef files should clean up generated C# projects

thanks for adding the issue and sorry I didn't read it before, will consider what you suggested.


Incompatible with fabricjs 1.6.x

Hi 😄
Tnx for this nice library you had implemented.
But, unfortunately it's not compatible with latest version of fabric js 1.6.x.
Maybe you can fix this issue easily by a little change, although I couldn't found a clue in your code!

For what it's worth, and for anyone else trying to figure this out, fabricjs-viewport works with fabricjs up through 1.6.0-rc.1 - after that, it fails silently. No idea why, either.

I def. needed something like this, why hasn't this gone up river?


Dictionary with DateTime as key is not supported

The serialization/deserialization of dictionaries is currently only possible with strings, numbers, or enums as keys. For transfering time-based data (e.g. for charts) I also need DateTime as key of the dictionary.
As long as I use the ISO8601 oder (milli-)seconds since unix epoch as date format a DateTime-key could be serialized as string or (quoted) number. Only the Newtonsoft-style milliseconds since unix epoch are a problem, since these would be serialized as Date-object, which would not be possible because the key gets the member/property name of the JSON object.
So I think DateTime as key of a dictionary should be possible as long as the date-format is not the Newtonsoft-style.
What do you think? Did I overlook something?
I am currently experimenting with a modified version of Jil which supports the scenario described above. With some additional cleanup and after extending the unit-tests for this new feature I could provide a PR for this.

DateTimes aren't supported as dictionary keys because they can't always be represented properly to be valid JSON.
Only Newtonsoft and ISO8601 styles are strings (a requirement for an object key, since as a rule Jil does not do implicit conversions [string-to-int-to-DateTime in the other DateTimeFormat cases]), I'm disinclined to make which type graphs are legal vary by the Options passed in.  That seems very counter-intuitive, and really really brittle for consumers.
Even if that weren't the case, the string-y DateTimes still make bad keys since they don't roundtrip with Tick granularity.  Newtonsoft-style is particular bad for that since it's only represented at millisecond granularity, but ISO8601 can also fail to roundtrip due to floating point rounding.  If Jil allowed these two DateTimes as keys, documentation would have to warn against looking up by key in a Dictionary which just feels wrong.
For time series data, I would instead suggest an array of objects.  This also has the benefit of being ordered, unlike a Dictionary, so the producer could force the events to be in the correct order for consumers.


Add ETL Contract Checks

Summary
Creates a series of methods on the ExtractTransformLoad base class that simplify and validate adherence to a standardized ETL contract for dataset sources. Fixes #617
Changes Proposed

Changes to data_pipeline/etl/base.py

Adds class GLOBAL CENSUS_CSV that can be used to read in the definitive list of Census Block Group FIPS codes from the census data
Creates an __init__() method to instantiate class attributes that can be used by each source ETL class and can be checked for compliance against the contract

self.NAME Used to create output path and populate logger info
self.SOURCE_URL Used to extract source data in self.extract()
self.OUTPUT_PATH Location where output csv is written in self.load()
self.SCORE_COLS List of columns that will be used in the score and should appear in the output csv
self.FIPS_CODES Dataframe of Census Block Group FIPS codes used to standardize the columns in the output csv

Create a private method self._get_census_fips_codes() which extracts the Census Block Group and Census Tract FIPS codes from the Census data and stores it in self.FIPS_CODES
Implements self._get_yaml_config() which parses a yaml file for the class and uses it to set class attributes in __init__()
Create an self.validate_output() method that can be optionally called to check the compliance of the output of ETL classes with the conditions below:

An output csv has been created in the data/dataset/{data_source_name} directory
The output csv has a column named GEOID10 which stores the census block group FIPS code
The columns in the output csv include the fields that will be used in the score and the names of those columns match the value expected by the score.
The GEOID10 column contains all of the Census block groups

Creates tests/base/ directory to test the new methods added to etl/base.py

Blockers or Questions

[ ] Several of these methods need to distinguish between the census ETL and the other dataset ETL classes, currently I've implemented this check by adding a key is_dataset to the config.yaml file. Is there a more elegant way to implement this check across class methods?
[ ] Are there other configuration variables we want to add to the config.yaml file for each dataset ETL? Or other class attributes that should be shared across dataset ETL classes?

Additional Context
A few of the features implemented in this PR are also designed to simplify the process of creating a "wizard" for new ETL classes, as described in #576, namely the pairing of self._get_yaml_config() with self.__init__() to dynamically populate a common set of class attributes that can be referenced by the other methods in the dataset classes.
Also, this PR contains the code for #549 Merging it will also merge that PR
Instructions to Review
TBD

@esfoobar-usds @switzersc-usds Here's the first slice of the code for the ETL checks. It looks like a lot of code, but only the last three commits are unique to this PR. The previous commits are from #549

Rebase to the last commit to main

Changes we discussed in today's office hours:

TODO: Change is_dataset to is_census
TODO: Rename geo_col to geoid_col

@switzersc-usds @esfoobar-usds Hey all! Thanks for your patience on this PR, it's been a busy couple of weeks, but I think this should be even with main and ready for review


dot autocompletion for "for"

When I am inside a "for" declaration and I want to write the three dots I receive an autocompletion for each dot. The perfect solution would be to add the three dots in one keystroke or ignore autocompletion in that context.

Hi Franco,
It should be fixed now :)
Cheers,
Clément


command order in makefile affects compilation for CUDA

When the makefile order is CUDA A - CUDA B - OpenACC A, the compilation is successful.

When the order is CUDA A - OpenACC A - CUDA B, the compilation fails
for both cases, i made cuda_PartB.cu's code the same as cuda_PartA.cu (greyscale implementation).

That was probably because the compiler option you set for c++ in $CMAKE_CXX_FLAGS, here is -acc will be propagated to cuda compiler nvcc. But it does not support this option. So the last compile sequence you show does not work.
If you really want to do it in order of cuda A - openacc - cuda B. You can either remove -acc option before compile cuda B or set the c++ compile flag temporarily by, instead of altering $CMAKE_CXX_FLAGS , doing target_compile_options(openacc_PartB PRIVATE acc). In this way, the option you set for c++ compiler will not tamper with nvcc. This propagation behavior probably comes from a global setting in the cmake config file for cuda.
I know this, because I encountered this a while ago, imao.


Add /api/v1/get-user/:userId

something to atleast add to the api otherwise than avatar

User system isn’t finalized.

Also planning on removing dark mode from being database stored and probably being client stored


RDP Wrapper under Windows 11

Does it work under Windows 11? I don't see it under "Supported Terminal Services versions:"

Issues section is for reporting bugs, for support and questions contact Telegram group chat.


[tool_crash] StateError: Bad state: No context with the running Dart application.

Command
flutter run -d chrome

Steps to Reproduce

...
...
...

Logs
StateError: Bad state: No context with the running Dart application.
#0      RemoteDebuggerExecutionContext.id (package:dwds/src/debugging/execution_context.dart:47:7)
<asynchronous suspension>
#1      AppInspector.contextId (package:dwds/src/debugging/inspector.dart:156:50)
#2      AppInspector._getExtensionRpcs (package:dwds/src/debugging/inspector.dart:497:26)
#3      AppInspector._initialize (package:dwds/src/debugging/inspector.dart:98:40)
<asynchronous suspension>
#4      AppInspector.initialize (package:dwds/src/debugging/inspector.dart:152:24)
#5      ChromeProxyService.createIsolate (package:dwds/src/services/chrome_proxy_service.dart:184:37)
<asynchronous suspension>
#6      ChromeProxyService.create (package:dwds/src/services/chrome_proxy_service.dart:151:23)
#7      DebugService.start (package:dwds/src/services/debug_service.dart:143:55)
#8      DevHandler._startLocalDebugService (package:dwds/src/handlers/dev_handler.dart:190:25)
#9      _rootRunUnary (dart:async/zone.dart:1198:47)
#10     _CustomZone.runUnary (dart:async/zone.dart:1100:19)
#11     _FutureListener.handleValue (dart:async/future_impl.dart:143:18)
#12     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:696:45)
#13     Future._propagateToListeners (dart:async/future_impl.dart:725:32)
#14     Future._completeWithValue (dart:async/future_impl.dart:529:5)
#15     Future._asyncCompleteWithValue.<anonymous closure> (dart:async/future_impl.dart:567:7)
#16     _rootRun (dart:async/zone.dart:1190:13)
#17     _CustomZone.run (dart:async/zone.dart:1093:19)
#18     _CustomZone.runGuarded (dart:async/zone.dart:997:7)
#19     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1037:23)
#20     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#21     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)

[✓] Flutter (Channel beta, 1.21.0-9.1.pre, on Linux, locale en_IN)
    • Flutter version 1.21.0-9.1.pre at /home/aman/snap/flutter/common/flutter
    • Framework revision be9bc8cb39 (3 days ago), 2020-08-18 09:46:08 -0700
    • Engine revision 267070c17a
    • Dart version 2.10.0 (build 2.10.0-7.2.beta)

[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /home/aman/Android/Sdk
    • Platform android-29, build-tools 28.0.3
    • ANDROID_HOME = /home/aman/Android/Sdk
    • Java binary at: /snap/android-studio/91/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[!] Android Studio (version 4.0)
    • Android Studio at /snap/android-studio/91/android-studio
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] Connected device (2 available)
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome 83.0.4103.116

! Doctor found issues in 2 categories.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: beta
Creation framework version: be9bc8cb3942bda5d8ef4e44b44616c470625e18

Hi @AmanMandloi
I see there's an open issue addressing the case you described (https://github.com/flutter/flutter/issues/60495).
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


update recipe to use latest osx-arm build

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as instructed in #21.
Here's a checklist to do before merging.

[x] Bump the build number if needed.
[ ] check if hdf7b55d_3_cpu is the osx-arm hash for libtvm

Fixes #21

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please rerender

@conda-forge-admin please rerender


[ignore]test memory usage

/deploy

This change is part of the following stack:

#356 ◀

Change managed by git-spice.

You can access the deployment of this PR at https://renku-ci-ds-356.dev.renku.ch

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

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.03%) to 90.42%

Files with Coverage Reduction
New Missed Lines
%

components/renku_data_services/connected_services/db.py
1
85.57%

components/renku_data_services/storage/blueprints.py
1
94.65%

components/renku_data_services/users/db.py
1
94.14%

Totals

Change from base Build<PHONE_NUMBER>0:
0.03%

Covered Lines:
9061

Relevant Lines:
10021

💛  - Coveralls

Tearing down the temporary RenkuLab deplyoment for this PR.


Blur images & panels thumbnails on plotting screen 'window1'

Blur images and panels thumbnails in Visit3.2.1 (x86_64.exe) appear at plotting screen window1, Dell Inspiron,  Windows 10. session file and screenshot attached:
blur_screen_VisIt321.zip
This behavior applicable to netcdf files tested
How should we adjust the setting to avoid preventing  bur on a screen ? or that is the bug to fix ?
Thanks
D

Dear Kathleen,
Thank you for sharing the thread   https://github.com/visit-dav/visit/discussions/12645.
Yes the topic is the same, while its seems the issue of blur on a plotting window was not resolved there.
Graphics driver is Intel (R) UHD Graphics  v <IP_ADDRESS>36 and  I've   turned off multiple display settings.

Thank you


Add validation to accountAddressCreate/accountAddressUpdate mutations

I want to merge this change because...
this PR will add the same address validation for accountAddressCreate/accountAddressUpdate mutations as for mutations from checkout.

Fixes #4729 #4484
Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

We should have benchmarks for all operations. But I'm fine with adding it in separate PRs.


Game freezes when loading into training area

Problem is that it's checking for sizes on the training dummy and failing because those fields don't exist.
Need to change it to
if config.large_monster_UI.monster_name_label.include.size and monster.size ~= nil then
if config.large_monster_UI.monster_name_label.include.silver_crown_threshold and config.large_monster_UI.monster_name_label.include.gold_crown_threshold and monster.small_border ~= nil and monster.big_border ~= nil then
elseif config.large_monster_UI.monster_name_label.include.silver_crown_threshold and monster.small_border ~= nil the
elseif config.large_monster_UI.monster_name_label.include.gold_crown_threshold and monster.big_border ~= nil then
on lines 2337, 2341, 2343, and 2345.
You might also want to add it to 2334 because the value is nil, but your string format takes a %s instead of %.0f so it doesn't crash. It just displays nil as the size.

Fixed in 60b67db2819ecd40a6641b6e55550fdd4dba0b14?


xds: Add support for send_all_clusters field in LRS.

This required updating the envoy-api submodule and two of its dependencies.

Known failures: #22523


WebClient class not following redirects

Steps to reproduce
$WebClient = New-Object "System.net.WebClient"
$path = $env:temp\wsusscn2.cab
$WebClient.DownloadFile("http://go.microsoft.com/fwlink/?linkid=74689", $path)

Expected behavior
To follow redirect and download file.
Actual behavior
MethodInvocationException:
Line |
  23 |      $WebClient.DownloadFile($URL,$Path)
     |      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Exception calling "DownloadFile" with "2" argument(s): "The remote server returned an error: (302) Moved Temporarily."

Environment data

Name                           Value
----                           -----
PSVersion                      7.0.0-rc.2
PSEdition                      Core
GitCommitId                    7.0.0-rc.2
OS                             Microsoft Windows 10.0.14393
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Perhaps this is related to https://github.com/PowerShell/PowerShell/issues/2896 ?

You use .Net class so it is not PowerShell issue. Please discuss in .Net repository.


sort Stormy-Eyed/Seeing in Apocyan/Haunted by Stairs below their base dream qualities

This also contains a fix to what I believe to a bug, which would lead to problems sorting Mirrorcatch Boxes (I think?). Please double-check that part.

Okay like this?


Inquiry Regarding GPU Memory and Number Requirements and Training Duration

I have a few questions regarding the computational resources required to do so.
GPU Memory Requirement: Could you please specify the amount of GPU memory required for training the models discussed in the paper? I would appreciate knowing if there are any specific GPU models that are recommended.
Number of GPUs: How many GPUs are needed to achieve the results presented in the paper? If it's scalable, could you provide a guideline on how the number of GPUs might affect the training process?
Training Duration: What is the approximate training time for the models using the recommended setup? Is this time frame based on a specific GPU configuration?

Hello @hero-y ,
We trained LGD on a cluster of three GPUs NVIDIA Quadro 4080 over 1000 epochs. Each GPU requires about 10-20GB RAM if I remember correctly. The training time was less than 3 days.

If it's scalable, could you provide a guideline on how the number of GPUs might affect the training process?

Yes, it is scalable, you can modify the number of GPUs by setting --nproc_per_node=n, where n is your desired number.


Image and other error handling

I was working with some slightly bad data and ran across a few error conditions to fix, plus added some general image 404 handling and a fix for the default head image when called from subdirectories (that is, people page schedules).  Also I added image handling for Grenadine's export data format, which is otherwise very close to Konopas' data format.

Useful stuff. I might tweak it so the Grenadine image link gets moved into links when the data is loaded, rather than handling multiple cases when rendering.
I've also discovered KonOpas accepts the image in either links.img or links.photo, so I'll probably follow suit (there are also links.url, links.fb, and links.twitter that I'll add in due course).

I think it would be better to parse the data into a separate, minimal, and preferably flat structure to pass around.  Grenadine has a lot of extra stuff in the json, and KonOpas' links are just extra structure with no upside.  (It was particularly confusing that sometimes the full person data was passed to Participants.js and sometimes only a short form.)

I think you are quite correct. I already flatten the name fields (if sent as arrays).
As you've seen, I copy fields from the People to the Program People. I've realised that it would be simpler to just replace the people in Program with references to the full People records. This would also be more memory efficient, since they would be just references to the same records rather than full copies of the fields.
I#m also finding my current method of breaking down the program into day and time slots untidy, requiring me to repeat components. I think it would be better to build a tree structure when initially loading the program, which would allow me to simplify the day and timeslot components.
I'm in the middle of a restructure that will incorporate these changes.

I've tidied up a bit in #45.
When processing people, looks for person.links.img, person.links.photo, or person.image_256_url. Puts whichever finds in new person.img field.
Puts reference to full person record in place of partial one in session.people.
I think you're probably right that creating new objects so that unused fields aren't carried over would be preferable in the long run. But this at least means the the Participant and Person components only need to look at one image field.


Dev to main sync

Developer Name: @MayurLund @heyrandhir
PR Number(s):

#1563
#1605

Issue Ticket Number:

#1418
#1606

Backend changes

[x] Yes
[ ] No

Frontend Changes

[x] Yes - On mysite and dashboard
[ ] No

Database changes

[x] Yes
[ ] No

Breaking changes (If your feature is breaking/missing something please mention pending tickets)

[ ] Yes
[x] No

Deployment notes(Any special instructions for the deployment)

[ ] Yes
[x] No

Tested in staging

[x] Yes
[ ] No

List of PRs going in this Sync:
-https://github.com/Real-Dev-Squad/website-my/pull/496
Proof of staging:
Verified the staging environment with the 'dev' flag, and everything's in order. I've provided post-update images for the frontend and dashboard. Additionally, I successfully initiated multiple extension requests; the latest one (No. 3) is currently pending. This pending status is preventing the creation of a new request. Here's a snapshot illustrating the pending request:

Resolved the conflicts. Please merge @iamitprakash @prakashchoudhary07. Thanks


Making handout work with Jupyter

First of all, thanks for creating handout!
Is there any plan to integrate Jupyter and Handout files in the future?
I think the combination of both would help solve lots of issues current notebooks have, while keeping the practical approach jupyter notebooks have!
I think if we could have in Python something similar to how Rstudio implemented notebooks (where you can easily commit text files) while keeping the flexibility of working interactively it would be a game changer!
Thanks again for your work!

Hi! Please see #25 and #33 and let me know if there are any function functionalities that you think would be useful in this regard. However, be aware that the premise of Python Handout is to write your code from a Python script rather than an interactive web interface.

Hi danijar thanks! I think the issues you linked cover different needs. For me the most useful way it would be to work as if I were using a Jupyter notebook, but having the advantages Handout offers (i.e. having the "notebook" code separated by the data and the artifacts needed to generate the output).
In this regard I think that Rmarkdown notebooks, as implemented by Rstudio, are the best compromise between the flexibility of interactive development and keeping the code under version control).
Thanks a lot again for your great work on Handout

Converting to and from Jupyter notebooks as discussed in #33 might help you. This way, you could work in Jupyter notebooks but commit Handout Python scripts in version control. It means you could also easily re-run a script without having to start Jupyter.


fix(versions): update Activiti/activiti-cloud-full-chart-security versions

UpdateBot pushed helm dependency: activiti-cloud-full-example to: 7.1.453

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-full-example 7.1.453


Incorrect GET request on fetcher.submit with method post

What version of Remix are you using?
1.19.2
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
Recently updgraded from remix 1.14.1 to 1.19.2
When a route has both loader and action exports, calling fetcher.submit with method post creates multiple network requests and is inconsistent.
It firsts makes correct POST request. and immediately a GET request is made to the same route which updates loaderdata and fetcher data twice and subsequent logic which relies on that data fails.
I've stripped down the code to minimal reproducible example,
Simply create an new remix example app
and update the code provided to see the behavior

Expected Behavior
It should not make GET request upon fetcher.submit
Actual Behavior
import { json, SerializeFrom, type V2_MetaFunction } from "@remix-run/node";
import { FetcherWithComponents, useFetcher } from "@remix-run/react";
import { useEffect, useState } from "react";

export const meta: V2_MetaFunction = () => {
  return [
    { title: "New Remix App" },
    { name: "description", content: "Welcome to Remix!" },
  ];
};

export const loader = async() => {
  return json("");
}

export async function action() {
  return json("")
}

export default function Index() {

  const fetcher = useFetcher();

  const startPolling = () => {
    console.log("CALLED")
    if (fetcher.state == "idle") {
      fetcher.submit({ _t : Date.now() },{ method: "post" });
    }
  }

  useEffect(() => {
    if (fetcher.state === "idle"){
      setTimeout(startPolling, 5000);
    }
  }, [fetcher]);

  return (

Welcome to Remix

  );
}

This is expected behavior, and isn't different between 1.14.1 and 1.19.2.  Loaders are always re-called after any submission (a process called "revalidation") to ensure that your UI reflects any changes made during the submission.  You can opt-out of this via the route level shouldRevalidate export: https://remix.run/docs/en/1.19.2/route/should-revalidate


Suspicious bug in logical form generation

Hi,
bert_seq2seq_reader.py: line 203 seems to be suspicious. Is it a bug or a feature?
Could it be  lfs_2 = generate_all_logical_forms_2(entities, offline=self._offline)?
It is somewhat unnatural to iterate through entities but always call generate_all_logical_forms_2 using entities[0].

Thanks for pointing it out! I think it was a design choice by me mainly because that for BERT-based model it is more time-consuming to generate candidate queries by iterating all the entities. I don't remember clearly how are the numbers of iterating all the entities compared with only using the first entity, but it is straightforward to try that because the ranking-based model is trained in the same way as the transduction-based model (which means candidate generation is not relevant to the training process), so we can directly do ranking on a different set of candidates during inference.

Thanks! I see the point. That makes sense. But line 203 will introduce duplicate logical forms anyway. Then, should it be moved out of the loop?
Btw, if we only generate the logical forms for the first candidate, does it make sense to also only call generate_all_logical_forms_alpha on entities[0] rather than iterating through all entities. (I mean delete line 201 and use entities[0] for line 202'.


oauth doesn't work in safari

I thought I had broken something because every time I would log in, it would just redirect me to the logged out page, but I realized it just didn't work in Safari and worked fine in chrome. I assume this is something wrong/incompatible in the storage part of oauth.

Hey! Quick question, are you using lockdown mode?

Hello,
It seems that this issue is related to vvo/iron-session#587.
As a possible solution, you can either host your application over HTTPS or set secure: false in your configuration.
I confirmed that by modifying https://github.com/bluesky-social/statusphere-example-app/blob/7077651a30159b646fab00061815dbbbb3e90c80/src/routes.ts#L41-L44 to:
const clientSession = await getIronSession<Session>(req, res, {
  cookieName: "sid",
  password: env.COOKIE_SECRET,
  cookieOptions: { secure: false },
});

it works in Safari.


[ROCm] fixed rocm_solvers build

this has updated rocm_solvers due to XLA gpu executor changes.

Hi @cheshire Can you please review this PR ? Thank you!

Hi @cheshire Can you please review this PR ? Thank you!

Hi @cheshire , Can you please review this PR ? Thank you!

Hi @cheshire , Can you please review this PR ? Thank you!

Hi @cheshire , Can you please review this PR ? Thank you!

Hi @cheshire , Can you please review this PR ? Thank you!

Hi @cheshire , Can you please review this PR ? Thank you!

Hi @i-chaochen, Can you please resolve the conflicts? Thank you !

Hi @i-chaochen, Can you please resolve the conflicts? Thank you !


Download multiple provisioning profiles matching specific certificate id

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
How to download multiple provisioning profiles at once from an account matching a specific certificate_id added to those provisioning profiles. sigh commands requires the bundle id to be specified which is unknown
Environment

 Gem                       | Version      |
| ------------------------- | ------------ |
| did_you_mean              | 1.2.0        |
| slack-notifier            | 2.3.2        |
| atomos                    | 0.1.2        |
| CFPropertyList            | 2.3.6        |
| claide                    | 1.0.2        |
| colored2                  | 3.1.2        |
| nanaimo                   | 0.2.3        |
| xcodeproj                 | 1.5.6        |
| rouge                     | 2.0.7        |
| xcpretty                  | 0.2.8        |
| terminal-notifier         | 1.8.0        |
| unicode-display_width     | 1.3.0        |
| terminal-table            | 1.8.0        |
| plist                     | 3.4.0        |
| public_suffix             | 2.0.5        |
| addressable               | 2.5.2        |
| multipart-post            | 2.0.0        |
| word_wrap                 | 1.0.0        |
| tty-screen                | 0.6.4        |
| tty-cursor                | 0.5.0        |
| tty-spinner               | 0.8.0        |
| babosa                    | 1.0.2        |
| colored                   | 1.2          |
| highline                  | 1.7.10       |
| commander-fastlane        | 4.4.6        |
| excon                     | 0.60.0       |
| faraday                   | 0.14.0       |
| unf_ext                   | <IP_ADDRESS>      |
| unf                       | 0.1.4        |
| domain_name               | 0.5.20170404 |
| http-cookie               | 1.0.3        |
| faraday-cookie_jar        | 0.0.6        |
| fastimage                 | 2.1.1        |
| json                      | 2.1.0        |
| mini_magick               | 4.5.1        |
| multi_json                | 1.13.1       |
| multi_xml                 | 0.6.0        |
| rubyzip                   | 1.2.1        |
| security                  | 0.1.3        |
| xcpretty-travis-formatter | 1.0.0        |
| dotenv                    | 2.2.1        |
| bundler                   | 1.16.1       |
| faraday_middleware        | 0.12.2       |
| uber                      | 0.1.0        |
| declarative               | 0.0.10       |
| declarative-option        | 0.1.0        |
| representable             | 3.0.4        |
| retriable                 | 3.1.1        |
| mime-types-data           | 3.2016.0521  |
| mime-types                | 3.1          |
| little-plugger            | 1.1.4        |
| logging                   | 2.2.2        |
| jwt                       | 2.1.0        |
| memoist                   | 0.16.0       |
| os                        | 0.9.6        |
| signet                    | 0.8.1        |
| googleauth                | 0.6.2        |
| httpclient                | 2.8.3        |
| google-api-client         | 0.13.6       |
| gh_inspector              | 1.1.3        |
| openssl                   | 2.1.0        |
| date                      | 1.0.0        |
| fileutils                 | 1.0.2        |
| ipaddr                    | 1.2.0        |

@amosingh There is no way to do it for matching a "certificate_id" right now. You have to either know the app_identifier or pull them all down and find your profile manually. This is not a highly requested feature so its not on our list of things to add right now but feel free to tackle this on your own and submit a PR 😊

Thank u very much. We found a work around.

Glad to hear! Happy fastlaning 🚀


when starting git gui in Powershell 5.1, next line shows ANSI codes

Environment
Windows build number: 10.0.18363.0
Windows Terminal version (if applicable): 0.7.3291 (and 0.6.2951.0)

Any other software?
Git for windows (v2.24.0.windows.2)
Windows Powershell v5.1

Steps to reproduce

On Windows Powershell prompt enter git gui
On next line start entering next command

Expected behavior

the entered text shows as it is supposed, with correct coloring from PSReadline.
Actual behavior

ANSI escape codes are showed until Enter is pressed.  The actual input is correctly executed.

When PSReadline module is removed from session, the problem ceases to exist.

This is a bad interaction between PSReadline and git -- one or the other is failing to restore the console mode to one that supports VT. Does it work with the new PSReadline verison in PowerShell 6/7?
(closing this as it's a shell/application issue)


[Bug]: Can't generate a PDF in desktop, tablet or mobile mode. Always in tablet !

Minimal, reproducible example
const http = require('http');
const puppeteer = require('puppeteer');
const devices = puppeteer.KnownDevices;

const PORT = process.env.PORT || 3000;

const server = http.createServer(async (req, res) => {
    try {
        // Extract the URL, filename, and device type from the request query parameters
        const url = new URL(req.url, `http://${req.headers.host}`).searchParams.get('url');
        const filename = new URL(req.url, `http://${req.headers.host}`).searchParams.get('filename');
        let deviceType = new URL(req.url, `http://${req.headers.host}`).searchParams.get('device');

        if (!url) {
            throw new Error('URL parameter is missing');
        }

        const browser = await puppeteer.launch({
          executablePath: "google-chrome",
          args: ['--no-sandbox', '--disable-setuid-sandbox']
        });

        const page = await browser.newPage();

        // Emulate device based on deviceType
        if (deviceType === 'phone') {
            await page.emulate(devices['iPhone X']); // Example for phone
        } else if (deviceType === 'desktop') {
            await page.setViewport({ width: 1280, height: 720 }); // Example for desktop
        } else {
            // Default or other types can be handled here, e.g., tablet
            await page.emulate(devices['iPad']); // Keeping your default tablet mode
        }

        page.setDefaultNavigationTimeout(0);

        await page.goto(url, { waitUntil: 'load' });

        // Generate PDF with additional options
        const pdf = await page.pdf({
            format: 'A4',
            printBackground: true,
            margin: { top: '20px', right: '20px', bottom: '20px', left: '20px' }
        });

        await browser.close();

        res.writeHead(200, { 'Content-Type': 'application/pdf', 'Content-Disposition': `filename=${filename}.pdf` });
        res.end(pdf);
    } catch (error) {
        console.error('Error:', error);
        res.writeHead(500, { 'Content-Type': 'text/plain' });
        res.end(error.message);
    }
});

server.listen(PORT, () => {
    console.log(`Server running on port ${PORT}`);
});

Error string
no error
Bug behavior

[ ] Flaky
[X] PDF

Background
I have an app where I pass a URL as argument and the script transform it to a PDF and return the PDF to the user.
Expectation
I have an app where I pass a URL as argument and the script transform it to a PDF and return the PDF to the user.
Reality
The problem is I can't change the way the PDF is displayed. It's always rendered in tablet mode. I've tried so different things and nothing helped.
If the URL is:

....&device=desktop : the PDF should represent what the html looks in desktop mode.
....&device=tablet : the PDF should represent what the html looks in tablet mode.
....&device=phone : the PDF should represent what the html looks in phone mode.

How can we achieve this?
Thanks.
Puppeteer configuration file (if used)
No response
Puppeteer version
22.0.0
Node version
20.11.0
Package manager
npm
Package manager version
8.5.5
Operating system
Windows

My question above is valid. It's running well, I can generate a PDF and it's well rendered. ONLY that I can't manage how to display the PDF in differents modes; desktop, tablet and phone.

@apinard could you please update the example to include a sample page URL and remove http server logic? Also, could you clarify what differences you expect in PDF rendering for different modes? As PDF rendering happens in the printing mode, it might not be affected by emulation.

Hi @OrKoN ,
Thank you for your fast reply.
I attached an example.

Basically, depending of the parameter in the url (desktop, tablet or phone), I could set internally a width/height or use a device like the initial idea.. but it's not working.
Any tips?
Thank you again.

@apinard the site is responsive and has probably print styles for PDF printing. Printing in the regular browser looks the same

So I don't think there is a bug.

@OrKoN ah ok, I didn't know there was the notion of "print styles". I'll take a look. Thank you for your fast answers.


anyone needs insert_into_file(between: ___) ?

Hi, I'm curious why nobody asking insert_into_file with between feature? I was get embarrassed many times without any solution to do this:
Here is a simplified webpacker.yml that we just focus on the key point.
# config/webpacker.yml
default: &default
  extract_css: false
development:
  <<: *default
test:
  <<: *default
production:
  <<: *default
  extract_css: true

I'm going to write a rails template that add extract_css: true to development section with this code:
inject_into_file 'config/webpacker.yml', after: /^development:\n *<<: \*default\n/ do
  <<~YAML.indent(2)
    extract_css: true
  YAML
end

unfortunately extract_css: true already exists after production section which also after development section, so inject_into_file will do nothing.
what if inject_into_file support option between or bi-options after + before, it should inserted successfully.
inject_into_file 'config/webpacker.yml', between: [/^development:\n *<<: \*default\n/, /^production:/] do
  <<~YAML.indent(2)
    extract_css: true
  YAML
end

or
inject_into_file 'config/webpacker.yml', after: /^development:\n *<<: \*default\n/, before: /^production:/ do
  <<~YAML.indent(2)
    extract_css: true
  YAML
end

My tricky solution is:
inject_into_file 'config/webpacker.yml', after: /^development:\n *<<: \*default\n/ do
  <<~YAML.indent(2)
    extract_css: true # this is for dev
  YAML
end


Tips for reducing/eliminating wear when using LittleFS?

I've been playing with the simulator and am impressed with the potential results vs FatFS. However, on some devices that are meant to be deployed for years at a time, a file system failure could mean total device failure. Is there some way to reset the filesystem (kind of like defragging it) to eliminate the wear on the device? Or would the only way to do this be moving all contents off the chip -> chip erase -> recreating the littlefs?

This simulator (http://littlefs.geky.net/demo.html?
In #660 it was mentioned "Ah, that simulation is unfortunately quite out of date. It's still using v1, which relied on bad-block reporting, but didn't provide true dynamic wear-leveling. v2 now has dynamic wear-leveling (which introduced the block_cycles configuration option.)"

Yeah I was using that simulator. I see it is out of date. I'll look more into the new dynamic wear-leveling. Thanks


Ender Dragon not paying or giving xp when using the hunter job

*Description of issue:
While my players are using the Hunter job, they are not being paid or receiving xp for doing so. I have set it in config and tried different variations such as - end dragon, ender dragon, ender drag etc but the issue still persists. As per previous recommendation through discord support, I have disabled dynamic payment and it worked for a day and the issue is still persisting. While we were testing, it works 'some' of the time but not every single time.
Server Type (Spigot/Paperspigot/etc):
Spigot
Server Version (using /ver):
1.15.2

The names should be ender_dragon. Also send here your Hunter job section.

Hunter:
fullname: Hunter
shortname: H
description: Earns money killing monsters.
 You need to check the generalConfig what you've been enabled before, and trying to disable that, like limitations.


Post in both on Instagram and Facebook

I know that this project was designed to work only with Instagram, but in the app, there is a feature to post publications and stories also on Facebook with a single checkbox (if the accounts are linked).
I think that would be awesome to have this funcionality in this project without the need to seek for the facebook API. It looked simple to me, but I dont know if it is hard to implement.

Please sponsor the development of this feature if you need it.
5 cups of coffee


Problems with 3D annotation in milestone #6

[x] Problem: Initial state of the loaded scene is useless

When an user goes to a frame he/she should see the full scene. I will recommend to use "bird eye view" (top view) because it is easy to recognize the road, cars and other objects. Now users have to zoom out always (unnecessary action). The reference tool is initialized by a side view where all points are visible. It is also an acceptable solution.

[ ] Problem: black and white points without colors

It was already a long story. Let me come back to the decision to show black and white points. The reference tool assigns different colors to a point depends on the distance from the ego-vehicle. Let me understand reasons why CVAT doesn't do that as well.

[x] Problem: Frame manipulation by mouse isn't activated by default in "select" mode

When a 3D task is loaded, it doesn't allow users to manipulate with the loaded 3D frame using mouse. It isn't how the reference tool works and CVAT 2D works. It isn't convenient.
Furthermore, if I choose "move" mode and go to another frame, "select" mode is activated and I can manipulate with the frame. Thus I can reproduce performance problems, which were reported previously.
AR: please fix performance problems, but don't break/disable expected features and functionality.

[x] Problem: Projection of points on side and front views is excess

Projection of points on "side" and "front" views is excess. Please look at the reference tool and compare the current implementation in CVAT and expected implementation in the reference tool. On the same PCD file, in CVAT it is mostly impossible to see boundaries of objects on "side" and "front" views. Looks like points which are far away of the 3D bounding box should not be displayed at all.

[x] Problem: Shape selection works with a significant delay

When I'm moving mouse under shapes, the selection works with a significant delay. I don't see the problem in the reference tool. Also, if I change "opacity" in the right side bar, UI is very unresponsive.

[ ] Problem: Cuboid can not be drawn in some cases

Start drawing a cuboid using a standard way, double-click a perspective view. UI is broken, cuboid cannot be drawn anymore, event in a new drawing session. Need to reload the page.

We can close it now


Design System: Consistent use of spacing, divider and typography

Problem to be solved
We want to apply consistent visual elements across the site to present the proper information hierarchy and make it easy to scan information for the users.
Design guidelines
https://zpl.io/aBqOM9L
Next steps

[x] @GarethOrmerod to audit and recommend guidelines
[x] @GarethOrmerod to review with @davidpmccormick
[ ] Dev to implement

Discussing this with @GarethOrmerod, it seemed feasible to try and work to a set of pre-defined spacings for various situations. At a first pass, these seemed like they could be:

breaks between slices in the body
breaks between 'sections' (header:body, body:stuff-below-the-body)
space within components

Sketch files so far have these spacings documented for desktop, but Gareth was keen for us to experiment on scaling them down uniformly for mobile.

@davidpmccormick I just noticed this on Events list. This list doesn't have any pagination. Therefore, it should have some padding between the components. Is this something to consider for the system spacing? https://wellcomecollection.org/events

I feel like section spacing is currently too big on mobile. @Heesoomoon what do you think?
For example, the space between the exhibition featured image and the quote is currently 78px:

I'd propose to reduce it to ~54px:


Add jmx endpoint to load new schema from disk

When performing a restore, this will allow us to load schema restored to system tables without restarting CassandraDaemon.

Closing in favor of #514


[OneExplorer] Add parent to Node

This commit adds parent member to Node.
This is required to implement 'reveal' in ONEExplorer. Because, ONE Explorer need to reveal it's all parent node to reveal the node consequently.
ONE-vscode-DCO-1.0-Signed-off-by: Dayoung Lee<EMAIL_ADDRESS>
From #1246

@PTAL :-D


implement Sacrifice to the Red God (TFM)

French spoiler: https://i.redditmedia.com/0uqmMRzViESffdRtQi59pPVqXFJde4cd88eXUA_VObA.jpg?w=495&s=2bfddfedf849902ed3ec0fa4aea3cf8e

this can now be merged, the official english card name has been spoiled on the Second Sons podcast and I've just updated the PR accordingly


Add starcoder2 (and dolphincoder) support to autocomplete (not complete yet)

This needed two changes:

Starcoder2 requires raw mode for ollama. The model template that ollama wraps around the prompt otherwise means FIM is not detected by the model
The file details passed to the autocomplete template needed to be structured, as starcoder2 needs them in a different format to what the fixed string encoding was (see https://arxiv.org/pdf/2402.19173.pdf for format details)

To do before this could be merged:

Change the file context to also be a structured object, rather than just a string. That needs changes to file-interaction cache.
Probably broke tests

And some other ideas to improve results generally:

Add language-specific stop words (which is how continue stops very long multi-line autocompletes)
A different idea would be to look at the depth in the tree hierarchy and stop if the autocomplete goes up a level (so if you're inside an if block, stop once the block has been completed at most)

BTW, I think there's a bug in the file-interaction code - onDidOpenTextDocument doesn't track focus or which window is active, multiple files can be "open" in different windows at the same time. I think it should be rewritten to use onDidChangeActiveTextEditor instead?
Raising for now to start discussion.

BTW, here's the list of extra stop-words that Continue uses per language. https://github.com/continuedev/continue/blob/main/core/autocomplete/languages.ts.

This is awesome thanks, is it ready for merge or are you adding more commits?
I realise that there are some improvements to make on FIM completions but I have not had the time to concentrate on it.  Those tests we're a bit brittle and a foundation so we can skip them if necessary for now to make improvements. I welcome more PRs from you if you have some improvements you would like to make.
Many thanks,

Yeah, I do notice starcoder2 doesn't know when to stop - or at least the 15B version I use. There's a note in the paper that they messed up the 15B FIM training, but the actual results (at least with the dolphincoder finetune) seem good. I'll test with 7B later. I have an idea for how to fix it anyway though.
(I still think deepseek-coder-33B gives better results, but you need different -base and -instruct versions and it's a much bigger model generally)

Since this PR is a month old, will close for now, many conflicts and things have changed.  Please reopen if willing to finish, many thanks.


Unable to import soundfile in AWS lambda

Trying to import soundfile inside AWS lambda results in the following error.
[ERROR] OSError: cannot load library 'libsndfile.so': /lib64/libm.so.6: version `GLIBC_2.27' not found (required by /opt/python/_soundfile_data/libsndfile.so)

Since I can't install linux packages inside lambda, I used the whl to install soundfile so that it includes it's own libsndfile and modified LD_LIBRARY_PATH to point to this libsndfile.so file.

The wheel is built and tagged for glibc >= 2.31 (because that's the oldest glibc available in Github's CI). You will need to supply your own libsndfile.

I can see libm-2.26.so inside my lambda's /lib64/ folder, where can I find the compatible libsndfile.so for this? Is there an older commit that you recommend I can use to build the wheel?

Sorry, this is not a tech support forum for AWS.

I just uploaded a wheel for glibc 2.17, which might solve your issue.


UnicodeDecodeError with command frame variable

Summary
In some cases, calling frame variables from Voltron causing the following stacktrace:
Traceback (most recent call last):
  File "<python>/site-packages/flask/app.py", line 1982, in wsgi_app
    response = self.full_dispatch_request()
  File "<python>/site-packages/flask/app.py", line 1614, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "<python>/site-packages/flask/app.py", line 1517, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "<python>/site-packages/flask/app.py", line 1612, in full_dispatch_request
    rv = self.dispatch_request()
  File "<python>/site-packages/flask/app.py", line 1598, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "<python>/site-packages/voltron/core.py", line 86, in api_post
    res = self.server.handle_request(request.data.decode('UTF-8'))
  File "<python>/site-packages/voltron/core.py", line 241, in handle_request
    res = self.dispatch_request(req)
  File "<python>/site-packages/voltron/core.py", line 297, in dispatch_request
    log.debug("Response: {}".format(str(res)))
  File "<python>/site-packages/voltron/api.py", line 193, in __str__
    return self.to_json()
  File "<python>/site-packages/voltron/api.py", line 234, in to_json
    return json.dumps(self.to_dict())
  File "<python>/json/__init__.py", line 243, in dumps
    return _default_encoder.encode(obj)
  File "<python>/json/encoder.py", line 207, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "<python>/json/encoder.py", line 270, in iterencode
    return _iterencode(o, 0)
UnicodeDecodeError: 'utf8' codec can't decode byte 0xf7 in position 147: invalid start byte

The line having the issues looks like
return json.dumps(self.to_dict())

I did a little bit of digging and it looks like the issue might be that the default encoding for json.dumps is "utf-8" but in this particular case it needs to be "iso-8859-1". I've found a temporary workaround of changing the offending line to:
return json.dumps(self.to_dict(), ensure_ascii=False)

Steps to Reproduce

Create a file char.cpp with the following contents:

struct s { char arr[2]; };

int main() {
  struct s foo = {{(char)247, '\0'}};
  return 0;
}

Compile with clang++ --debug char.cpp to generate a.out
Start up the debugger with lldb a.out
Set a breakpoint right before the program finished with breakpoint set -l 5
Launch the program with process launch
In a new window view the frame variables with voltron by running voltron view command "frame variable". This will cause the error.

Expected results
Step 6 should not error.
Environment
clang++: Apple LLVM version 8.1.0 (clang-802.0.42)
python: Python 2.7.10 :: Anaconda custom (x86_64)
voltron: 0.1.7
voltron-web: 0.1.1
lldb: lldb-370.0.42
os: macOS 10.12.5

For an example that's easier to mess around with you can also use the Python REPL:
$ python
>>> import json
>>> dict = {'status': 'success', 'data': {'output': '(s) foo = (arr = "\xf7)'}, 'type': 'response'}
>>> json.dumps(dict)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "python2.7/json/__init__.py", line 243, in dumps
    return _default_encoder.encode(obj)
  File "python2.7/json/encoder.py", line 207, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "python2.7/json/encoder.py", line 270, in iterencode
    return _iterencode(o, 0)
UnicodeDecodeError: 'utf8' codec can't decode byte 0xf7 in position 18: invalid start byte

I ran into this same issue and used your workaround as well.
The following command output can't be properly decoded:
dispatch_request -- Response: {"status": "success", "data": {"output": "(i32) a = 0\n(const char *) path = 0x0000000000024580 \"UH\\x89H\u0005\""}, "type": "response"}
which causes UnicodeDecodeError: 'utf8' codec can't decode byte 0xe5 in position 60: invalid continuation byte.
The code I'm debugging is like
{
    int a = 0;
    const char* path = "foo";
}

and if I set a breakpoint on the first line "frame variable" in lldb will output
(i32) a = 0
(const char *) path = 0x0000000000024580 "UH\x89H"

due to the fact that path hasn't been initialized yet and is just garbled memory, which voltron tries to make python decode as utf8.


Develop

PR for CMM intro and tutorial to be merged into master.

Is it possible to get a preview of the website for the changes in this PR?


Packet len is out of range when starting

when consumer starting , it will get all dubbo node from zookeeper, then "Packet len is out of range"
why not only get  node that with @DubboReference
just for less request ?
    void readLength() throws IOException {
        int len = this.incomingBuffer.getInt();
        if (len >= 0 && len < this.packetLen) {
            this.incomingBuffer = ByteBuffer.allocate(len);
        } else {
            throw new IOException("Packet len" + len + " is out of range!");
        }
    }

Environment

Dubbo version:  2.7.8
Operating System version: xxx
Java version: 8

Steps to reproduce this issue

xxx
xxx
xxx

Pls. provide [GitHub address] to reproduce this issue.
Expected Result
What do you expected from the above steps？
Actual Result
What actually happens?
If there is an exception, please attach the exception trace:
Just put your stack trace here!

There are two solutions:

You can increase the jute.maxbuffe parameter and try it
Use application-level service registration and discovery

@szd1993 You can refer to the above answer, temporarily close the issues, if there are other issues, you can reopen the issues.


How to get the access token?

Pleas provide more information. What are you trying to achieve?

Sorry! I managed to get the access token. It would be much helpful if you can add an example of how user will login to facebook using this package.
Also is there any way to set the access token like 'vinkla/instagram' package, so that we do not need to add it in every api call?
https://github.com/vinkla/instagram#examples
We can set the access token once
Instagram::setAccessToken($data);

Yeah, you can set it in the configuration with default_access_token .


Add implicit '/' to directories

Something like this to fix #22

Thanks for this, it gets me a little closer to what I want out of this project. However if I enable directory listings, this feature no longer works. I really need to learn Go...but any chance you can extend this PR so it takes precedence over directory listings?

Ah sorry, I've just released the dockerfile was building from git and not my local checked out changes. Your patch may indeed do what I need if I can work out how to build in into an image.

This seems to do what I was looking for, but is probably terrible code...
diff --git a/main.go b/main.go
index 85e64dd..4867758 100644
--- a/main.go
+++ b/main.go
@@ -314,19 +314,20 @@ func awss3(w http.ResponseWriter, r *http.Request) {
                }
                path = link.URL + path[idx+12:]
        }
+       var pathToGet string
        if strings.HasSuffix(path, "/") {
-               if c.directoryListing {
-                       s3listFiles(w, r, c.s3Bucket, c.s3KeyPrefix+path)
-                       return
-               }
-               path += c.indexDocument
+               pathToGet = c.s3KeyPrefix+path+c.indexDocument
+       } else {
+               pathToGet = c.s3KeyPrefix+path
        }
-       obj, err := s3get(c.s3Bucket, c.s3KeyPrefix+path, rangeHeader)
+       obj, err := s3get(c.s3Bucket, pathToGet, rangeHeader)
        if aerr, ok := err.(awserr.Error); ok {
                switch aerr.Code() {
                case s3.ErrCodeNoSuchKey:
-                       path += "/" + c.indexDocument
-                       obj, err = s3get(c.s3Bucket, c.s3KeyPrefix+path, rangeHeader)
+                       if c.directoryListing {
+                               s3listFiles(w, r, c.s3Bucket, c.s3KeyPrefix+path)
+                               return
+                       }
                }
        }

          pathToGet = c.s3KeyPrefix+path+c.indexDocument

What about:
filepath.Join(c.s3KeyPrefix, path, c.indexDocument)

?
It's more safe from "//" point of view.
Of course, S3 allows you to have paths with double slash, but it's definitely not ordinary case.

@antonu17 I like this very simple suggestion. It actually would solve a challenge running behind ngnix for me too. Sad, that it has not been merged.

@pottava Any update on this PR? Is there anything to be done?


attribute all copied code

We should make sure that we correctly attribute copied code before publishing this.

copypasta deleted via #12


[MISC] Consolidate cleanup() and refactor offline_inference_with_prefix.py

When refactoring offline_inference_with_prefix.py due to #9505, I found that cleanup() has been implemented many times in contests. This PR consolidates them to a utility function.
cc @tlrmchlsmth @DarkLight1337 @youkaichao

CI failed due to #9424 which is unrelated


Description must be prefixed with WEB - Info box should be more prominent

At the bottom of the page is a "note" box that tells you:

Notes description must be prefixed with WEB ('WEB' keyword with an asterisk sign (*) before and after) in order to display on the portal.

It should be given more prominence. For example, place it at the top of the page underneath the note that tells you that you need to give explicit entity permissions.

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

ID: 434c988b-243e-0d01-e098-fa9a0300ae31
Version Independent ID: e7b47bad-3195-2b6d-bef3-8c23b6fb58e8
Content: Configure notes as attachments on entity forms and web forms for a portal - Power Apps
Content Source: powerapps-docs/maker/portals/configure-notes.md
Service: powerapps
GitHub Login: @GitanjaliSingh33msft
Microsoft Alias: gisingh

Thank you for reaching out, documentation has been updated.


Reduce typing of element file names

See original issue on GitLab
In GitLab by [Gitlab user @ptomato] on Aug 24, 2017, 00:08
It would be nice if the element's .bst extension were implicit in commands like bst build or bst track. (If you use element-path in project.conf, then you can't do tab-completion from the current path, so you have to type out the whole thing.

In GitLab by [Gitlab user @tristanvb] on Aug 24, 2017, 00:15
Not sure about doing this one...
I have a click branch where i've improved the automated bash completions significantly enough to handle buildstream's CLI well but it was not merged there (think its a case of overworked maintainers).
With the above it will be trivial enough to check if a -C option was specified and do a shallow parse of project.conf to get the element-path, and just use that to complete bst files (also the rest of the completion mechanics are also very luxurious).
Instead i may just port that code directly into buildstream and i think that may obviate the need for this... Right ?

In GitLab by [Gitlab user @ptomato] on Aug 24, 2017, 01:41
I think that would work.

In GitLab by [Gitlab user @tristanvb] on Aug 28, 2017, 19:09
Completions are now implemented in master.
Unfortunately integration is not completely automated with an installation of buildstream from git using pip but should be automated with distro packages in some future.
Instructions have been added to the documentation and these are visible from the main docs page and table of contents.

In GitLab by [Gitlab user @tristanvb] on Aug 28, 2017, 19:09
closed

In GitLab by [Gitlab user @samthursfield] on Sep 20, 2017, 16:21
mentioned in commit ddcff898aac546a7b85e62eea0552f4166076f20

In GitLab by [Gitlab user @samthursfield] on Sep 20, 2017, 17:16
mentioned in commit 9c83378250ecc6c62b5c32909640d64fd6888d3f

In GitLab by [Gitlab user @samthursfield] on Sep 27, 2017, 13:21
mentioned in commit bf0f2863e8cea73e41d5d9efca3126e4075fc3ae


Bump bindgen to 0.55

Thank you @N3xed . I've published the updated version https://crates.io/crates/leptonica-sys/0.3.3


AutoAppove not working as expected

Hi,
basically I do not really get how the AutoAppove Feature is working.
What I want to do is, to automatically approve the Baseline Images.
The general setup is the following. I take screenshots before the deployment, which should function as the new baseline (These should be AutoApproved). I deploy. And then I do the run the check for changes.
So I tried to always use a new baseline branch. Then I do not get AutoApprove - only new marked screens, that need to be manually approved. And I tried to set the threshold for the baseline run to 100%. There it is only set as passed and I cant even approve the images manually. If I do it normally, changing images need to be approved. But that is to be expected.
How does this feature exactly work. I did not find an explanation here.
I am using the Cypress Agent.
Thanks!

Just found the wiki. Will look into.


v23.1.6 release notes

Preview:
https://deploy-preview-17519--cockroachdb-docs.netlify.app/docs/releases/v23.1#v23-1-6

Hi @ashleykwtam !
Thanks for link to the preview - that was super helpful!

The first 3 fixes[1] listed should not be included

I.e. the only PRs / release notes to include should come from these PRs:

https://github.com/cockroachdb/cockroach/pull/106863
https://github.com/cockroachdb/cockroach/pull/106426
https://github.com/cockroachdb/cockroach/pull/107105

(which maps to these commits: https://github.com/cockroachdb/cockroach/compare/v23.1.5...staging-v23.1.6)
Here's a quick check that I did to look at the release notes from 106863, 106426, 107105, in case that's helpful:

https://docs.google.com/document/d/15ZfdSUkub3NcFuyAop9Dqf88AFARYH7NrSR03dKacTA/edit

[1] #105830, #105899, #105925 (in pink), should be removed for this change log.


GH-37470: [Python][Parquet] Add missing arguments to ParquetFileWriteOptions

Rationale for this change
I think this may have been missed when this feature was added.

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Closes: #37470

Would you mind add a test for write_page_index? Since we need to ensure page index would be written.

I've added a basic test for this.

Great! I'll take a careful round later. By the way we can disable statistics to just disable Column Index but write offset index. Maybe this could be add to doc later.

Great! I'll take a careful round later. By the way we can disable statistics to just disable Column Index but write offset index. Maybe this could be add to doc later.

Are you saying write_statistics=False, wrtie_page_index=True will write the OffsetIndex but not the ColumnIndex? If that's the case then I'd absolutely agree the docs should be updated as currently they suggest that write_statistics is ignored if write_page_index=True! But this should probably be separate as the docs will need updating in pyarrow.parquet too?

Great! I'll take a careful round later. By the way we can disable statistics to just disable Column Index but write offset index. Maybe this could be add to doc later.

Are you saying write_statistics=False, wrtie_page_index=True will write the OffsetIndex but not the ColumnIndex? If that's the case then I'd absolutely agree the docs should be updated as currently they suggest that write_statistics is ignored if write_page_index=True! But this should probably be a separate PR as the docs will need updating in pyarrow.parquet too?

That is indeed what you were saying as the test shows! That behaviour strikes me as quite unintuitive!

That is indeed what you were saying as the test shows! That behaviour strikes me as quite unintuitive!

Cause I'm the author of that behavior lol
When disable collect statistics, currently it would be hard to collect the ColumnIndex, because ColumnIndex relies on Statistics.
Also, by the way, by default, if column index is enabled, the page header statistics will not be written. (Since spec says if column index exists, page header is not tent to be written)

When disable collect statistics, currently it would be hard to collect the ColumnIndex, because ColumnIndex relies on Statistics.

Yeah, that makes sense! Would be good to clarify in the docs - will see if I get to it in another PR.

Also, by the way, by default, if column index is enabled, the page header statistics will not be written. (Since spec says if column index exists, page header is not tent to be written)

Yeah, what gets written and what doesn't is quite confusing. In fact the spec doesn't techincally say that the page-level statistics when writing the ColumnIndex but that it isn't recommended (one might want both in order to support old readers).
https://github.com/apache/parquet-format/blob/master/PageIndex.md#technical-approach
It's probably sensible default behaviour but it'd be nice to force being able to write both.

The logic is ok here, however, I think we should export the flag to parquet/core.py, and from that to _dataset_parquet.pyx. You can take a look at your another patch ( #37665 ). Since just a _dataset_parquet config is hard to use for users.

The flags are already there! They were just never added to the ParquetFileWriteOptions:
https://github.com/apache/arrow/blob/e56726c161df1d588f2f6197dbdd0c0011f14855/python/pyarrow/parquet/core.py#L976

One thought: would it be worth adding docstrings for ParquetFileWriteOptions and adding examples of use to the Python user guide? As a separate issue?

Any updates? Should we rerun continuous-integration/appveyor/pr and checkin if it's a unrelated flacky test?

Any updates? Should we rerun continuous-integration/appveyor/pr and checkin if it's a unrelated flacky test?

The failing AppVeyor test would be fixed with rebasing the branch (https://github.com/apache/arrow/issues/37555).
It would be good to see if something can be done regarding the documentation (see) before merging.

Any updates? Should we rerun continuous-integration/appveyor/pr and checkin if it's a unrelated flacky test?

Yes, please - if it doesn't pass on a second run I'll look into it today.

Any updates? Should we rerun continuous-integration/appveyor/pr and checkin if it's a unrelated flacky test?

The failing AppVeyor test would be fixed with rebasing the branch (#37555).

Thanks, I've done this now.

It would be good to see if something can be done regarding the documentation (see) before merging.

I can try to make time to get to this but it feels like a pretty independent issue - especially as the WriteOptions classes don't even appear in the Python docs: https://arrow.apache.org/docs/python/api/dataset.html#classes. It feels like it would make more sense to add documentation for IPC, CSV, ORC, and Parquet FileWriteOptions all in one PR?

I can try to make time to get to this but it feels like a pretty independent issue - especially as the WriteOptions classes don't even appear in the Python docs: https://arrow.apache.org/docs/python/api/dataset.html#classes. It feels like it would make more sense to add documentation for IPC, CSV, ORC, and Parquet FileWriteOptions all in one PR?

Agree totally, it deserves a special issue for sure! What you propose sounds great, thank you!
It would be enough for now just to open a separate issue and link it from here, so we do not forget.

Agree totally, it deserves a special issue for sure! What you propose sounds great, thank you! It would be enough for now just to open a separate issue and link it from here, so we do not forget.

#37800

All test passed now, can we try to checkin this patch? :-)

Great, will merge 👍