text:
using existing stable diffusion instalation

Hi bro,
If we can use already existing stable diffusion(for eg: Automatic1111) it will save a lot of disk space.
Just a suggestion.
Thank you

Hi,
Thank you for the suggestion, Automatic1111 did not have Depth2IMG back then, I think it does now, so this could be possible.
But probably using ControlNet could be a great upgrade, so I hope to test switching backend to one of then some time in the future.

That's some conflict of packages, try:
conda install -c anaconda openssl
after conda activate


text:
Implement the Elo Algorithm

Investigate to implement the Elo Algorithm to the matchmaking system
https://en.wikipedia.org/wiki/Elo_rating_system#Other_usage

@pH-7 this is very interesting. Nice!

Tinder no longer uses the Elo algorithm because it didn't work for them. (Google it if you don't believe me.) If you wanna take an actually algorithmic approach to this, I'd suggest maybe starting with this paper:
https://arxiv.org/abs/1905.07342

Tinder no longer uses the Elo algorithm because it didn't work for them. (Google it if you don't believe me.) If you wanna take an actually algorithmic approach to this, I'd suggest maybe starting with this paper:
https://arxiv.org/abs/1905.07342

Interesting indeed! Thanks very much for sharing this paper @no-identd
Any further suggestions are always very welcome 🤗

source: gharchive/issue

authors: Undefined-Variables no-identd pH-7

license: MIT

license_type: permissive

license_source: github-api


text:
Clients does not seem to be initialized anymore in DefaultCallbackLogic

Hi,
In the past, the DefaultCallbackLogic would call Clients.findClient to find a client.
This would initialize the Clients object if needed and all the clients configured.
Now there is this DefaultCallbackClientFinder used and it doesn't seem to do this initialization anymore.
I think that's a bug, no? I suppose it was caught before because usually people would first go through the DefaultSecurityLogic (which does initialize the Clients object via DefaultSecurityClientFinder's calls to Clients.findClient) first. But with jax-rs-pac4j, it happens that someone would directly go through a DefaultCallbackLogic.

You must be right, the behavior has changed. I will fix that.

authors: leleuj victornoel


text:
Fix --split

This fixed my issue as reported in #38

output filename changed from result["accession"] to result, as keyed is a dictionary with string keys
output filename content changed accordingly to keyed[result]

No probs and thanks for the tool and the future updates :)

Hi @telatin please go ahead and submit your PR directly against the #devel branch as we have made some bug fixes there. Once you've submitted your code changes (along with a unit test case) then we can go ahead and merge into #devel.

I see some tests are failing (like 500 server error). I imagine it's a nightmare to manage this tool :)

It is definitely challenging. What's strange is that the tests past locally but not via automated testing in GitHub Actions. We've not been able to diagnose that yet..

OK- the tests are all working now. Please pull the latest version from devel!

I'll close out this PR so that a new one can be made against devel. Thank you again for your contributions!

source: gharchive/pull-request

authors: sbooeshaghi telatin

repo: pachterlab/ffq


text:
gget.cellxgene TileDBError error when trying to return anndata

What happened?
When running gget.cellxgene(     species="homo_sapiens", meta_only=False, census_version="2023-05-15", column_names=COLUMN_NAMES,     assay=PROTOCOLS, is_primary_data=True )
I get the error "RuntimeError: TileDBError: in operator syntax must be written as `attr in ['l', 'i', 's', 't']"
If I instead specify meta_only=True, it runs fine.
Here is the full output/error message:
Tue Nov  7 11:12:20 2023 INFO Fetching AnnData object from CZ CELLxGENE Discover. This might take a few minutes...
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
Cell In[18], line 1
----> 1 gget.cellxgene(
      2     species="homo_sapiens", meta_only=False, census_version="2023-05-15", column_names=COLUMN_NAMES,
      3     assay=PROTOCOLS, is_primary_data=True
      4 )

File /opt/conda/rpeyser/envs/scset/lib/python3.8/site-packages/gget/gget_cellxgene.py:188, in cellxgene(species, gene, ensembl, column_names, meta_only, tissue, cell_type, development_stage, disease, sex, is_primary_data, dataset_id, tissue_general_ontology_term_id, tissue_general, assay_ontology_term_id, assay, cell_type_ontology_term_id, development_stage_ontology_term_id, disease_ontology_term_id, donor_id, self_reported_ethnicity_ontology_term_id, self_reported_ethnicity, sex_ontology_term_id, suspension_type, tissue_ontology_term_id, census_version, verbose, out)
    184     logging.info(
    185         "Fetching AnnData object from CZ CELLxGENE Discover. This might take a few minutes..."
    186     )
    187 with cellxgene_census.open_soma(census_version=census_version) as census:
--> 188     adata = cellxgene_census.get_anndata(
    189         census=census,
    190         organism=species,
    191         var_value_filter=f"{'feature_id' if ensembl else 'feature_name'} in {gene}",
    192         obs_value_filter=obs_value_filter,
    193         column_names={"obs": column_names},
    194     )
    196     if out:
    197         adata.write(out)

File /opt/conda/rpeyser/envs/scset/lib/python3.8/site-packages/cellxgene_census/_get_anndata.py:81, in get_anndata(census, organism, measurement_name, X_name, X_layers, obs_value_filter, obs_coords, var_value_filter, var_coords, column_names)
     75 var_coords = (slice(None),) if var_coords is None else (var_coords,)
     76 with exp.axis_query(
     77     measurement_name,
     78     obs_query=soma.AxisQuery(value_filter=obs_value_filter, coords=obs_coords),
     79     var_query=soma.AxisQuery(value_filter=var_value_filter, coords=var_coords),
     80 ) as query:
---> 81     return query.to_anndata(X_name=X_name, column_names=column_names, X_layers=X_layers)

File /opt/conda/rpeyser/envs/scset/lib/python3.8/site-packages/somacore/query/query.py:263, in ExperimentAxisQuery.to_anndata(self, X_name, column_names, X_layers)
    244 def to_anndata(
    245     self,
    246     X_name: str,
   (...)
    249     X_layers: Sequence[str] = (),
    250 ) -> anndata.AnnData:
    251     """
    252     Executes the query and return result as an ``AnnData`` in-memory object.
    253
   (...)
    261     Lifecycle: maturing
    262     """
--> 263     return self._read(
    264         X_name,
    265         column_names=column_names or AxisColumnNames(obs=None, var=None),
    266         X_layers=X_layers,
    267     ).to_anndata()

File /opt/conda/rpeyser/envs/scset/lib/python3.8/site-packages/somacore/query/query.py:335, in ExperimentAxisQuery._read(self, X_name, column_names, X_layers)
    332         raise NotImplementedError("Dense array unsupported")
    333     all_x_arrays[_xname] = x_array
--> 335 obs_table, var_table = self._read_both_axes(column_names)
    337 x_matrices = {
    338     _xname: _fast_csr.read_scipy_csr(
    339         all_x_arrays[_xname], self.obs_joinids(), self.var_joinids()
    340     )
    341     for _xname in all_x_arrays
    342 }
    344 x = x_matrices.pop(X_name)

File /opt/conda/rpeyser/envs/scset/lib/python3.8/site-packages/somacore/query/query.py:362, in ExperimentAxisQuery._read_both_axes(self, column_names)
    352 obs_ft = self._threadpool.submit(
    353     self._read_axis_dataframe,
    354     _Axis.OBS,
    355     column_names,
    356 )
    357 var_ft = self._threadpool.submit(
    358     self._read_axis_dataframe,
    359     _Axis.VAR,
    360     column_names,
    361 )
--> 362 return obs_ft.result(), var_ft.result()

File /opt/conda/rpeyser/envs/scset/lib/python3.8/concurrent/futures/_base.py:437, in Future.result(self, timeout)
    435     raise CancelledError()
    436 elif self._state == FINISHED:
--> 437     return self.__get_result()
    439 self._condition.wait(timeout)
    441 if self._state in [CANCELLED, CANCELLED_AND_NOTIFIED]:

File /opt/conda/rpeyser/envs/scset/lib/python3.8/concurrent/futures/_base.py:389, in Future.__get_result(self)
    387 if self._exception:
    388     try:
--> 389         raise self._exception
    390     finally:
    391         # Break a reference cycle with the exception in self._exception
    392         self = None

File /opt/conda/rpeyser/envs/scset/lib/python3.8/concurrent/futures/thread.py:57, in _WorkItem.run(self)
     54     return
     56 try:
---> 57     result = self.fn(*self.args, **self.kwargs)
     58 except BaseException as exc:
     59     self.future.set_exception(exc)

File /opt/conda/rpeyser/envs/scset/lib/python3.8/site-packages/somacore/query/query.py:396, in ExperimentAxisQuery._read_axis_dataframe(self, axis, axis_column_names)
    393     added_soma_joinid_to_columns = True
    395 # Do the actual query.
--> 396 arrow_table = axis_df.read(
    397     axis_query.coords,
    398     value_filter=axis_query.value_filter,
    399     column_names=query_columns,
    400 ).concat()
    402 # Update the cache if needed. We can do this because no matter what
    403 # other columns are queried for, the contents of the ``soma_joinid``
    404 # column will be the same and can be safely stored.
    405 if not joinids_cached:

File /opt/conda/rpeyser/envs/scset/lib/python3.8/site-packages/tiledbsoma/_dataframe.py:341, in DataFrame.read(***failed resolving arguments***)
    338 if value_filter is not None:
    339     query_condition = QueryCondition(value_filter)
--> 341 sr = self._soma_reader(
    342     schema=schema,  # query_condition needs this
    343     column_names=column_names,
    344     query_condition=query_condition,
    345     result_order=result_order,
    346 )
    348 self._set_reader_coords(sr, coords)
    350 # TODO: platform_config
    351 # TODO: batch_size

File /opt/conda/rpeyser/envs/scset/lib/python3.8/site-packages/tiledbsoma/_tiledb_array.py:118, in TileDBArray._soma_reader(self, schema, column_names, query_condition, result_order)
    116     result_order_enum = result_order_map[ResultOrder(result_order).value]
    117     kwargs["result_order"] = result_order_enum
--> 118 return clib.SOMAArray(
    119     self.uri,
    120     name=f"{self} reader",
    121     platform_config=self._ctx.config().dict(),
    122     timestamp=(0, self.tiledb_timestamp_ms),
    123     **kwargs,
    124 )

RuntimeError: TileDBError: `in` operator syntax must be written as `attr in ['l', 'i', 's', 't']`

At:
  /opt/conda/rpeyser/envs/scset/lib/python3.8/site-packages/tiledbsoma/_query_condition.py(213): visit_Compare
  /opt/conda/rpeyser/envs/scset/lib/python3.8/ast.py(371): visit
  /opt/conda/rpeyser/envs/scset/lib/python3.8/site-packages/tiledbsoma/_query_condition.py(133): init_query_condition
  /opt/conda/rpeyser/envs/scset/lib/python3.8/site-packages/tiledbsoma/_tiledb_array.py(118): _soma_reader
  /opt/conda/rpeyser/envs/scset/lib/python3.8/site-packages/tiledbsoma/_dataframe.py(341): read
  /opt/conda/rpeyser/envs/scset/lib/python3.8/site-packages/somacore/query/query.py(396): _read_axis_dataframe
  /opt/conda/rpeyser/envs/scset/lib/python3.8/concurrent/futures/thread.py(57): run
  /opt/conda/rpeyser/envs/scset/lib/python3.8/concurrent/futures/thread.py(80): _worker
  /opt/conda/rpeyser/envs/scset/lib/python3.8/threading.py(870): run
  /opt/conda/rpeyser/envs/scset/lib/python3.8/threading.py(932): _bootstrap_inner
  /opt/conda/rpeyser/envs/scset/lib/python3.8/threading.py(890): _bootstrap

gget version
0.27.9
Operating System (OS)
Linux
User interface
Python
Are you using a computer with an Apple M1 chip?
Not M1
What is the exact command that was run?
gget.cellxgene(
    species="homo_sapiens", meta_only=False, census_version="2023-05-15", column_names=COLUMN_NAMES,
    assay=PROTOCOLS, is_primary_data=True
)

Which output/error did you get?
RuntimeError: TileDBError: `in` operator syntax must be written as `attr in ['l', 'i', 's', 't']

Hi Rebecca, thank you for reaching out. Could you please provide your COLUMN_NAMES and PROTOCOLS variables? I would like to reproduce this error so I can hopefully figure out what's going on.

Thanks for looking into it! Let me know if you don't reproduce, and I can send more details about my environment setup.
PROTOCOLS = [
"10x 5' v2",
"10x 3' v3",
"10x 3' v2",
"10x 5' v1",
"10x 3' v1",
"10x 3' transcription profiling",
"10x 5' transcription profiling"
]
COLUMN_NAMES = [
"soma_joinid",
"is_primary_data",
"dataset_id",
"donor_id",
"assay",
"cell_type",
"development_stage",
"sex",
"disease",
"tissue",
"tissue_general"
]

Hi, I just pushed a possible fix to the main branch. If you install gget from source you should be able to test it out. Please let me know if you try it! If this works, it will be part of the next release (v0.28.0).
To install gget from source run the following commands from the terminal:

Clone the gget repo:
git clone https://github.com/pachterlab/gget.git
Navigate into the gget folder and use pip to install the local version:
cd gget && pip install .

authors: lauraluebbert rpeys

repo: pachterlab/gget


text:
Fix a bug where error is not returned when primary key is not found

Next() returns false when there's an error.  So it's when Next() is false that you want to return cursor.Err().

I have a record number of two one-character PRs at this point I think.

This is a pretty lulz bug right here. I think the behavior there was technically correct.

authors: derekchiang jdoliner

repo: pachyderm/pachyderm


text:
packit.yaml: create downstream %changelog from git-log

Hunor is correct, our upstream release description used as a %changelog
violates Fedora guidelines, so let packit assemble %changelog from
git-log.
https://docs.fedoraproject.org/en-US/packaging-guidelines/#changelogs
https://packit.dev/docs/configuration/#copy_upstream_release_description

I could have link the downstream PRs here, sorry about that:
https://src.fedoraproject.org/rpms/python-ogr/pull-request/255#request_diff
 * Fri Feb 19 2021 Packit Service <user-cont-team+packit-service@redhat.com> - 0.21.0-1
 # 0.21.0

 ## Features

 - Implemented `get_files` for Pagure projects (by [@mfocko](https://github.com/mfocko)).

 ## Minor

 - Some other change...

The leading # are the worst because rpm treats those as comments.

The leading # are the worst because rpm treats those as comments.

That's ogr, not packit, I don't see anything like that in packit downstream changelog.
Do the guidelines say that we can't have comments or empty lines in there?
(I'm nitpicking because you're saying that we're violating the rules, but I don't see which one)
Hope we can start using %autochangelog soon.

Changelog entries should provide a brief summary of the changes done to the package between releases, including noting updating to a new version, adding a patch, fixing other spec sections, note bugs fixed, and CVE’s if any. They must never simply contain an entire copy of the source CHANGELOG entries. The intent is to give the user a hint as to what changed in a package update without overwhelming them with the technical details. Links to upstream changelogs can be entered for those who want additional information.

Fedora Packaging Guidelines

Thanks, so I see "They must never simply contain an entire copy of the source CHANGELOG entries." - which is what we were doing, OK, fair enough.
But how does this PR improve it? Now we're dumping a huge list of upstream commits into the downstream changelog, which is IMHO even worse than "copy of the source CHANGELOG entries" because the guidelines also say "without overwhelming them with the technical details".
IMHO the best would be just a simple "New upstream release 1.2.3", nothing else, because as I understand that paragraph, the downstream changelog should contain downstream changes, not upstream changes.

Yeah, I agree with this: the currently available features to automatically update changelogs are pretty limited. We would need a more comprehensive approach, in order to be able to meet all the wishes. @lachmanfrantisek what do you think?

Thanks, so I see "They must never simply contain an entire copy of the source CHANGELOG entries." - which is what we were doing, OK, fair enough.
But how does this PR improve it? Now we're dumping a huge list of upstream commits into the downstream changelog, which is IMHO even worse than "copy of the source CHANGELOG entries" because the guidelines also say "without overwhelming them with the technical details".
IMHO the best would be just a simple "New upstream release 1.2.3", nothing else, because as I understand that paragraph, the downstream changelog should contain downstream changes, not upstream changes.

Well, this is hard.
Here is an example how git-log style %changelog could look:
* Wed Nov 25 2020 Packit Service <user-cont-team+packit-service@redhat.com> - 0.21.0-1
--
- pre-commit autoupdate (Jiri Popelka)
- 0.21.0 release (Release bot)
- parsing git remote URL: inform what's happening... (Tomas Tomecek)
- Revert "Allow recursive search for specfile in repository" (Matej Focko)
- Regenerate test_data for recursive (Matej Focko)
- Allow recursive search for specfile in repository (Matej Focko)
- cli.copr-build: replace / with - (Tomas Tomecek)
- copr, log CoprException.result when creating repo fails (Tomas Tomecek)
- Delete recipe-tests.yaml (Jiri Popelka)
- Add build to default jobs (lbarcziova)
- Add test case for Upstream._fix_spec_source() (Nikola Forró)
- Fix SpecFile.get_source() (Nikola Forró)

Since we are releasing often, we never have 250 commits in a release, so I
wouldn't call such %changelog entries too overwhelming, though most of the
entries are probably useless to users.
The thing is that all projects and teams do rpm %changelog differently and
there is literally no best practice:

rebase-helper does the simple new upstream release

cockpit and anaconda post actual changes in the functionality: https://src.fedoraproject.org/rpms/anaconda/blob/rawhide/f/anaconda.spec#_424 https://src.fedoraproject.org/rpms/anaconda/blob/rawhide/f/anaconda.spec#_424

If we want people to read our upstream release
notes and just say "new upstream release x.y.z" that's perfectly fine: let's
agree as a team and implement the solution. If we want our downstream
changelogs to be more meaningful, we can do git-log style as you can see
above or implement something more complex to cherry-pick lines from the release
notes.
The way packit does %changelogs right now is based on the feedback from
anaconda and cockpit teams, hence the current situation.
I'm personally leaning towards more meaningful changelog though the most
important question is: are people reading RPM %changelogs and what would they
like to find there?
To me, rpm %changelog is an outdated implementation of a problem which is
really hard to solve - inform users about changes which are impactful to them.

I am not 100% sure if we are discussing packit's behaviour or the packaging of packit itself.
My notes to this:

I don't read the changelog of other packages. (Only of the applications if that is the spec changelog shown in gnome-software.) So I can't speak of the people that do.
If we are discussing packit itself, do you think we need to fulfil "They must never simply contain an entire copy of the source CHANGELOG entries." entirely? Since we are the upstream developers and we can make GitHub release notes in any format, what about putting there information that can be reused in Fedora? We can make it short, meaningful and without formatting problems.
Using git-log is better than nothing (=new upstream release), but too detailed for me.

authors: TomasTomecek csomh jpopelka lachmanfrantisek

repo: packit/packit


text:
Skip the performance test plan on dist-gt PRs

It was intended to run on upstream PRs only and it fails on EPEL8 because of old curl version. Just skip it.

/packit-stg build

/packit-stg build

/packit-stg build

repo: packit/specfile


text:
refactor: removed deprecated LockWallet and UnLockWallet from WalletService

Remove deprecated LockWallet and UnLockWallet methods from WalletService
Description

Deprecated Definition LockWallet & ``UnLockWallet are removed.
gRPC Implementation for LockWallet & ``UnLockWallet that were returning unimplemented status are removed.

Related issue(s)

Fixes #1340

Thanks @ansh-devs

repo: pactus-project/pactus


text:
Use "logger" module for log analytics

The module currently creates its log analytics resources by itself. It could benedif from using the terraform-azurerm-logger module instead

Resolved by #11

repo: padok-team/terraform-azurerm-network


text:
minor error in documentation for the blog tutorial

In the blog tutorial admin-dashboard-setup portion, there is a minor punctuation error.
Currently it looks like this
$ bundle exec rake db:migrate'
$ bundle exec rake db:seed'

which does not work and should be the following instead:
$ bundle exec rake db:migrate

and then
$ bundle exec rake db:seed

Please note the removed apostrophe and that the commands are separated for readability

Fixed in https://github.com/padrino/padrino-docs/commit/1c325c547b304302c17e02fb15362930ec03fcb9

authors: mketiku ujifgc

repo: padrino/padrino-framework


text:
🛑 School of intersession is down

In b222b48, School of intersession (https://www.schoolofintercession.eu) was down:

HTTP code: 0
Response time: 0 ms

Resolved: School of intersession is back up in 713aeaa after 15 minutes.

repo: pageon/uptime


text:
🛑 evadoc is down

In 65e0be8, evadoc (http://evadoc.be) was down:

HTTP code: 500
Response time: 1533 ms

Resolved: evadoc is back up in 2f8e815 after 19 hours, 8 minutes.


text:
mod_pagespeed directives in htaccess are ignored when requesting a php file

Hello,
First thank you for this amazing module,
I am facing an issue while adding directive to .htaccess related to mod page speed, they get completely ignored but only when requesting a php file, they work normally if the requested file is an html file.
the .htaccess only contains
<IfModule pagespeed_module>
  ModPagespeed off
</IfModule>

the module is always loaded for .php files and is turned off for .html file
Please advise.
Regards,

Looks like this was reported earlier via https://github.com/pagespeed/mod_pagespeed/issues/963
Closing this as a duplicate

authors: Tarekajaj oschaaf

license_source: bigquery


text:
ERROR: Invalid inlined_image_type in cached_result

Hello,
Here is the error full log entry:
[ngx_pagespeed 1.11.33.4-0] [0412/125341:ERROR:image_rewrite_filter.cc(1950)] Invalid inlined_image_type in cached_result
Here is the configuration of the PS module:
pagespeed EnableFilters prioritize_critical_css;
pagespeed EnableFilters defer_javascript;
pagespeed EnableFilters sprite_images;
pagespeed EnableFilters rewrite_images;
pagespeed EnableFilters recompress_png;
pagespeed EnableFilters convert_png_to_jpeg,convert_jpeg_to_webp;
pagespeed EnableFilters collapse_whitespace,remove_comments;
pagespeed EnableFilters convert_to_webp_animated;
pagespeed EnableFilters insert_dns_prefetch;
pagespeed EnableFilters lazyload_images;
pagespeed EnableFilters convert_to_webp_lossless;
pagespeed LazyloadImagesAfterOnload off;

The NGINX is version 1.10.3 build with dynamic modules and without debug.
If inline_images filter is disabled, then the error does not appear.
Any insight on what might be the cause of this?

Do you have any animated webp images referenced in the html that triggers the error getting logged?
The code that logs this error looks like:
  if ((image_type_value < IMAGE_UNKNOWN) ||
      (image_type_value > IMAGE_WEBP_LOSSLESS_OR_ALPHA)) {
    // IMAGE_UNKNOWN and IMAGE_WEBP_LOSSLESS_OR_ALPHA must be the smallest
    // and largest values, respectively, in ImageType enum.
    LOG(DFATAL) << "Invalid inlined_image_type in cached_result";
    return INLINE_INTERNAL_ERROR;
  }

(https://github.com/pagespeed/mod_pagespeed/blob/1.11.33.4/net/instaweb/rewriter/image_rewrite_filter.cc#L1950)
ImageType is defined as
enum ImageType {
  IMAGE_UNKNOWN = 0,
  IMAGE_JPEG = 1,
  IMAGE_PNG = 2,
  IMAGE_GIF = 3,
  IMAGE_WEBP = 4,
  IMAGE_WEBP_LOSSLESS_OR_ALPHA = 5,
  IMAGE_WEBP_ANIMATED = 6
};

Ooh, nice catch. Probably should just be using  ImageType_MIN and  ImageType_MAX
On Wed, Apr 12, 2017 at 12:14 PM, Otto van der Schaaf
notifications@github.com wrote:

Do you have any animated webp images referenced in the html that triggers
the error getting logged?
The code that logs this error looks like:
if ((image_type_value < IMAGE_UNKNOWN) ||
(image_type_value > IMAGE_WEBP_LOSSLESS_OR_ALPHA)) {
// IMAGE_UNKNOWN and IMAGE_WEBP_LOSSLESS_OR_ALPHA must be the smallest
// and largest values, respectively, in ImageType enum.
LOG(DFATAL) << "Invalid inlined_image_type in cached_result";
return INLINE_INTERNAL_ERROR;
}
(https://github.com/pagespeed/mod_pagespeed/blob/1.11.33.4/net/instaweb/rewriter/image_rewrite_filter.cc#L1950)
ImageType is defined as
enum ImageType {
IMAGE_UNKNOWN = 0,
IMAGE_JPEG = 1,
IMAGE_PNG = 2,
IMAGE_GIF = 3,
IMAGE_WEBP = 4,
IMAGE_WEBP_LOSSLESS_OR_ALPHA = 5,
IMAGE_WEBP_ANIMATED = 6
};
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

authors: ic1011 morlovich oschaaf


text:
Just added api_key.txt

that's pretty much it :)

just done the same :)

repo: paichiwo/world-weather


text:
Enable support for node & yarn detection

Summary
This PR adds support for detection of a package.json or yarn.lock, which will cause the following buildpacks/dependencies to be required as part of the build:

yarn.lock - requires install of yarn and node dependencies (can be provided by the yarn & node-engine buildpacks)
package.json - requires install of node dependency (can be provided by the node-engine buildpack)

The buildpack will not look for these files and request Node/Yarn installation if the Maven build is being performed by another buildpack, i.e., the Maven buildpack is only providing the installation of Maven.
To enable support for detection of these files, users should set the environment variable BP_JAVA_INSTALL_NODE to true (defaults to false to prevent false-positive node builds).
The buildpack will check the application root for the package.json or yarn.lock file. Users can set variables supported by the NodeEngine buildpack, such as BP_NODE_PROJECT_PATH, which can contain a sub-directory within the app root to search for these files.
Full implementation will require both the Yarn and NodeEngine buildpacks to be added to the Java composite order group as optional.
Use Cases
Enables projects which build frontend/javascript code to run commands such as 'yarn' to build alongside Java code. See paketo-buildpacks/java#651.
Checklist

[ ] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[ ] I have reviewed the styleguide for guidance on my code quality.
[ ] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Bump 🙂
Please let me know if there's anything I can help with.

authors: TisVictress pivotal-david-osullivan

repo: paketo-buildpacks/maven


text:
Don't fail when linux ca-bundle file does exist for Palantir CAs. Also support centos

Before this PR
Internally, a service has a build that uses a redhat based linux rather than ubuntu. It does not have /etc/ssl/certs/ca-certificates.crt. Instead it has:
$ ls -l /etc/ssl/certs/
total 16
lrwxrwxrwx 1 root root   49 Sep 30  2021 ca-bundle.crt -> /etc/pki/ca-trust/extracted/pem/tls-ca-bundle.pem
lrwxrwxrwx 1 root root   55 Sep 30  2021 ca-bundle.trust.crt -> /etc/pki/ca-trust/extracted/openssl/ca-bundle.trust.crt

It fails with an error that:
Caused by: java.nio.file.NoSuchFileException: /etc/ssl/certs/ca-certificates.crt
	at com.palantir.gradle.jdks.PalantirCaPlugin.linuxSystemCertificates(PalantirCaPlugin.java:124)

After this PR
==COMMIT_MSG==

Support reading Palantir CA from the truststores of Redhat based systems
Do not fail if the system truststore cannot be found on linux
==COMMIT_MSG==

Possible downsides?

Does Files.exists check whether the link exists or the underlying file?

👍 was on a phone when I reviewed, but it does follow links, LGTM

Released 0.30.0

authors: CRogers felixdesouza svc-autorelease

repo: palantir/gradle-jdks


text:
Improve detection mechanism

Currently, the program identifies vulnerabilities by matching based on file names and classes. However, this could be improved by analyzing the content to detect cases in which JARs are shaded and/or slightly modified by frameworks such as ProGuard.

Roughly:

md5 hashing of entire .class file contents to match against published log4j versions
md5 hashing of bytecode instructions within .class files to match shaded classes where the package has been changed but everything else remains the same
partial matching of bytecode instructions to handle obfuscation or more aggressive shading where e.g. unused methods are deleted

The first of these is now done, with the second two left to do.

https://github.com/palantir/log4j-sniffer/pull/27 implements the second of these 3.

authors: hpryce nmiyake


text:
no-unused-variable false positive when let/const is declared in a require statement

this code incorrectly reports that fs is unused:
let fs = require("fs"); // failure

fs.readFileSync("test.ts");

replacing let with const results in the same failure.

Duplicate of #325, which is due to a problem in language services.

I just tried it out and this bug + #325 are fixed with TS v1.5.3

authors: adidahiya leeavital

repo: palantir/tslint


text:
Bump NPM dependencies and switch to ESLint

fixes #1220

Also, build errors because of invalid peer dependencies :/

yep, noticed the invalid peer deps... will try to investigate tomorrow

Related: https://github.com/unindented/grunt-npm-command/issues/7
I think that might solve the peerDependency issues

I'd actually prefer to remove the grunt-npm-command dependency. It is a small package with very few users.

Sounds good, as long as things work correctly across platforms/CIs. Note https://github.com/palantir/tslint/pull/1172 which is the relevant discussion when it was added

grunt-npm-command released a new version to fix the peerDep error, so the build is passing now. I'm inclined to just let it stay for now, it's working well in our build.

meh, ok. filed follow-up #1235

authors: JKillian adidahiya


text:
Fix #2986 await-promise accept intersection

PR checklist

[x] Addresses an existing issue: #2986
[x] New feature, bugfix, or enhancement

[x] Includes tests

[ ] Documentation update

Overview of change:
Rule await-promise accepts not only union type but intersection type.
Is there anything you'd like reviewers to focus on?

CHANGELOG.md entry:

Thanks for your interest in palantir/tslint, @yokomotod! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

@adidahiya @ajafff Thank you for your reviews.
sorry, I didn't notice that my build has been failed... It pass the test now!

authors: palantirtech yokomotod


text:
Labels can be used with for-of statements.

Added for-of statement to the label position rule whitelist.

Thanks for the PR @ChaseMoskal! Could you add a test to test/rules/label-position/test.ts.lint? In this case, you just need to add a labeled for-of statement to the file (this would cause tests to fail without your change, but with your change tests should pass).

I think...
...I think I did it right.
Right?

lgtm :+1:

authors: ChaseMoskal JKillian adidahiya


text:
Create Supporting Administrative Information

Not really an issue, but trying to use GitHub to organize ideas...
One of the EarthCube committees put together a really great Community Participation Guidelines.

I renamed this issue to reflect the broader sets of documentation we have to create.

[ ] Community Participation Guidelines
[ ] Software Licenses
[ ] Data Licenses
[ ] Executive committee by-laws

Any others?

This is definitely a good document to use as a guide.

https://www.geosociety.org/gsa/membership/code-of-ethics/gsa/membership/code-of-ethics.aspx

@ecurrano This is in progress now. We should probably come up with a list of tasks and prioritize after HackAThon.

Activities related to this were explicitly proposed in the failed FY24 geoinformatics proposal. Because the FY25 proposal will be a partnership with PBDB, which has its own governance structure, we would have to be significantly revise this anyway. Closing for now and we can open a new issue if/when 2.0 happens.

authors: aazaff clairecleveland ecurrano

repo: paleobot/pbot-dev


text:
PBot Explore: Collection Search Results

Here is a screenshot of what the "Quick Results" would look like for a collections search. We envisioned this in table format, but this may not be phone/tablet compatible. So, I could also see the information we want to show for each collection just listed underneath the collection name.

For min and max ages: for standardization on this quick view results, converting everything to international epoch would be preferred. However, if that is too difficult, we could display whatever timescale was entered for min and max by the user.
When people click on a collection, they would see the full search results, and we love the way it is set up now.
The Show me more data button is a placeholder to allow for data downloads.

Some of the summary results is implemented in
paleobot/pbot-client@11aeab475d9bb0daeb1236e32c813a3c7683cab0
merged to master and running on dev

Location (lat/lon) search is in
paleobot/pbot-api@652479b8a184ba3ba1a6b66e56dc413c58c874a8
paleobot/pbot-client@
paleobot/pbot-client@
paleobot/pbot-client@

Location (lat/lon) search is in
https://github.com/paleobot/pbot-api/commit/652479b8a184ba3ba1a6b66e56dc413c58c874a8
paleobot/pbot-client@1e362dc9dd9d57d4ac6bb18fbb2d0c4a304ccf71
paleobot/pbot-client@dc09931443d71efcf4a0979c95ff72235679552d
Merged to master and running on dev.
Note 1: To create the required location property in the Collection nodes, this cypher was run before turning this loose:
match (c:Collection  )
set
    c.location = Point({latitude: c.lat, longitude: c.lon})
return c

Note 2: The api changes were made in a way that should maintain continuity with the previous. In other words, you can access the deprecated lat and lon properties as well as the new location.latitude/location.longitude The lat and lon properties accessed through the api are generated on the fly from those inside location.
Note 3: Ideally, we would also delete the old lat and lon properties in the nodes. I'm leaving them around for now in case there are any latent problems with the conversion cypher in note 1. However, they are no longer accessible through the api (see note 2).

Note: 4: The search radius is currently set to 1500 meters. Let me know if we want it bigger/smaller.

Ah hah! I started playing about with what the radius was. We definitely want this bigger but Dori, Claire, Andrew, and I should consult on this. Maybe a whole degree? In v2, it might be nice to allow the user to specify a radius- I could see wanting a small radius for some questions but a big one for others. To some extent, you can get around the big radius now by doing a search by country or state.

Also, I think the quick search results look great. I'd prioritize getting other things done over standardizing min and max age or getting the number of specimens or OTUs at a site. Those would be icing on the cake.

My comments regarding location search were meant for #283. I've moved them there for clarity. Ellen, you might want to move your relevant comment as well so we track the discussion there.

In the collection record page, under references: when multiple references, there is nothing separating the different references, not even a space. Possible to have each reference as separate lines, like for specimens?

This may be too much work for the amount of time we have left, but it would be very helpful to add an accordion for OTUs to be able to show all the OTUs present at the site.

@ecurrano, I agree - I like that there is a list of linkable specimens, but a list of the OTUs identified at the site would be really useful as well.
I thought we could get that info a different way by searching OTUs and just refine by collection, but I just tried and realized that you can't search by a specific collection, just location/time.

There's no way to search by collection id? That's bad.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: doricon @.>
Sent: Wednesday, September 25, 2024 9:23:09 AM
To: paleobot/pbot-dev @.>
Cc: Zaffos, Andrew - (azaffos) @.>; Assign @.>
Subject: [EXT] Re: [paleobot/pbot-dev] PBot Explore: Collection Search Results (Issue #287)
External Email

@ecurranohttps://github.com/ecurrano, I agree - I like that there is a list of linkable specimens, but a list of the OTUs identified at the site would be really useful as well.
I thought we could get that info a different way by searching OTUs and just refine by collection, but I just tried and realized that you can't search by a specific collection, just location/time.
—
Reply to this email directly, view it on GitHubhttps://github.com/paleobot/pbot-dev/issues/287#issuecomment-2374548946, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACPQSQ325QRY5EII4LS4FHLZYLPO3AVCNFSM6AAAAABELLYJ6GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZUGU2DQOJUGY.
You are receiving this because you were assigned.Message ID: @.***>

authors: NoisyFlowers aazaff doricon ecurrano


text:
Mongoengine Detail view skips embedded documents

The mongoengine edit page supports editing embedded documents directly, but the detail page does not.
Being able to show embedded document without the editor would be really useful

This is also needed feature.. No works ?

I am closing this issue as we've removed support for flask-mongoengine, because that library is no longer being maintained and is incompatible with recent versions of our core dependencies (eg flask, wtforms, etc).

authors: ptxmac samuelhwilliams ufo911

repo: pallets-eco/flask-admin


text:
Mongoengine Support in Version 2

Hello team,
First of, thank you very much for flask-admin.
As already mentioned, in this ticket, my tool heavily depends on mongoengine.
After some first proof of concepts, I found that migration to pymongo would be a huge step backwards. It's for my program, which basically consists of DB Models like a rewrite with fewer features and comfort after. (Example Filters also missing)
As a replacement for flask-mongoengine I found flask-mongoeine-3 which works well, and can replace the old one without any changes of code.
for flask-admin I was able with just some lines of code to make the contrib/mongoengine work again.
And my Plan B would to just ship that part with my Project. But since I'm sure others could use it too,
I want to ask why It can't stay as part of the flask-admin? I need to fix it anyway for me to keep it working, so I could also help with Pull Requests.  I just would need to change everything in order that the tests work again. But the function is already there by changing two imports.
Best
Bastian

Thanks for the update. Disappointing to hear that pymongo doesn't look like a feasible replacement.
flask-mongoengine-3 is broadly an option. My concern is that it appears to quite inactive as far as maintenance goes.
I'm not really against it per se, but bear in mind I have no personal motivation either as I don't use mongo in my day-to-day so it's not work I'd be heavily invested in.
Flask-Admin is a community maintained project so I think it would be very possible for you to look at adding support for flask-mongoengine-3 back (which hopefully may not be much harder than reverting the PR which took it away and making whatever tweaks/changes are needed for -3 support).
Maybe join the Pallets discord and check out the #pallets-eco channel? I'm happy to look at PRs you put up adding support (back) for it.

Hello,
My Flask app uses pure MongoEngine and Flask-Admin, and it works perfectly, thanks! Maybe there's an option to remove the Flask-MongoEngine dependency but keep MongoEngine.
As far as I can see, the current version of Flask-Admin uses only the orm and fields functionality from Flask-MongoEngine in the form.py file, but everything else is from MongoEngine.
Additionally, the orm module is deprecated in Flask-MongoEngine.

As far as I can see, the current version of Flask-Admin uses only the orm and fields functionality from Flask-MongoEngine in the form.py file, but everything else is from MongoEngine.

But isn’t that a crucial part of the code, given its responsibility for the WTF Forms? Otherwise, I could start exploring alternatives, possibly by shifting the related logic from flask_mongoengine into the flask_admin module.

Good idea to remove the requirement completely, will test that. It just will take me some days to start, much to do currently.

Good idea to remove the requirement completely, will test that. It just will take me some days to start, much to do currently.  Thank you very much already.

I've just tried to replace Flask-MongoEngine imports with MongoEngine and WTForms and it's working fine for me.

If this is workable, it sounds like a very positive idea and alleviates concerns around maintenance issues with flask-mongoengine and its forks 👍

@karpitsky I just tested your fixed version of forms.py and It works fine. Perfect

Ok, found one Problem. ReferenceFields stopped working.
If nested in a List, they just can't find a matching entry.
But when used like name = db.ReferenceField(),
there is a View Error in FlaskAdmin:
Traceback (most recent call last):
  File "/usr/local/lib/python3.12/site-packages/flask_admin/base.py", line 369, in _run_view
    return fn(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask_admin/model/base.py", line 2093, in create_view
    form = self.create_form()
           ^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask_admin/model/base.py", line 1332, in create_form
    return self._create_form_class(get_form_data(), obj=obj)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/wtforms/form.py", line 209, in __call__
    return type.__call__(cls, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask_admin/form/__init__.py", line 22, in __init__
    super(BaseForm, self).__init__(formdata=formdata, obj=obj, prefix=prefix, **kwargs)
  File "/usr/local/lib/python3.12/site-packages/wtforms/form.py", line 281, in __init__
    super().__init__(self._unbound_fields, meta=meta_obj, prefix=prefix)
  File "/usr/local/lib/python3.12/site-packages/wtforms/form.py", line 49, in __init__
    field = meta.bind_field(self, unbound_field, options)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/wtforms/meta.py", line 28, in bind_field
    return unbound_field.bind(form=form, **options)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/wtforms/fields/core.py", line 387, in bind
    return self.field_class(*self.args, **kw)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/wtforms/fields/choices.py", line 85, in __init__
    super().__init__(label, validators, **kwargs)
  File "/usr/local/lib/python3.12/site-packages/wtforms/fields/choices.py", line 25, in __init__
    super().__init__(label, validators, **kwargs)
TypeError: Field.__init__() got an unexpected keyword argument 'allow_blank'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.12/site-packages/flask/app.py", line 1498, in __call__
    return self.wsgi_app(environ, start_response)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask/app.py", line 1476, in wsgi_app
    response = self.handle_exception(e)
               ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask_restx/api.py", line 672, in error_router
    return original_handler(e)
           ^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask/app.py", line 1473, in wsgi_app
    response = self.full_dispatch_request()
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask/app.py", line 882, in full_dispatch_request
    rv = self.handle_user_exception(e)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask_restx/api.py", line 672, in error_router
    return original_handler(e)
           ^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask/app.py", line 880, in full_dispatch_request
    rv = self.dispatch_request()
         ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask/app.py", line 865, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)  # type: ignore[no-any-return]
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask_admin/base.py", line 69, in inner
    return self._run_view(f, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask_admin/base.py", line 371, in _run_view
    return fn(cls=self, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^
TypeError: BaseModelView.create_view() got an unexpected keyword argument 'cls'

I made out:
TypeError: Field.__init__() got an unexpected keyword argument 'allow_blank'

so i Changed:
    def conv_Reference(self, model, field, kwargs):
        #kwargs['allow_blank'] = not field.required

Which fixed the error, but still no entries show up.

Check this commit out, it should fix the issue.

authors: Bastian-Kuhn karpitsky samuelhwilliams


text:
Flask Quickstart on master branch; minimal example fails with NameError

I pulled the latest version of flask Flask from the master branch, commit 64ba43411fbfb2e435455b0df2453d7809b7c5a8.
The Quickstart minimal example on the dev version of Flask does not run as intended.  Previous versions of the Quickstart example would display a simple "Hello world" message, but the addition of an undefined "name" variable causes the example to fail with a NameError.
It seems that the intention was to illustrate Werkzeug's HTML escaping functionality early-on, which makes sense, but the example needs to be restructured or augmented so that it still runs.
I'm aware that this is a development version of Flask, so my apologies if this (possibly known?) issue should be reported elsewhere or can be ignored.  Thanks.
Expected Behavior
The Quickstart doc says that running the example should display a "Hello world" message when navigating to http://127.0.0.1:5000/.
from flask import Flask
from markupsafe import escape

app = Flask(__name__)

@app.route("/")
def hello_world():
    return f"
Hello, {escape(name)}!
"

Actual Behavior
The browser displays an Internal Server Error response from Flask due to a NameError.
Traceback (most recent call last):
  File "/home/pkritikos/work/flask_playground/venv/lib/python3.6/site-packages/flask/app.py", line 2430, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/pkritikos/work/flask_playground/venv/lib/python3.6/site-packages/flask/app.py", line 1945, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/pkritikos/work/flask_playground/venv/lib/python3.6/site-packages/flask/app.py", line 1814, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/pkritikos/work/flask_playground/venv/lib/python3.6/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/home/pkritikos/work/flask_playground/venv/lib/python3.6/site-packages/flask/app.py", line 1943, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/pkritikos/work/flask_playground/venv/lib/python3.6/site-packages/flask/app.py", line 1929, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/pkritikos/work/flask_playground/tutorial/__init__.py", line 8, in hello_world
    return f"
Hello, {escape(name)}!
"

Environment

Python version: Python 3.6.9
Flask version: Flask 1.2.0.dev0
Werkzeug version: Werkzeug 1.0.0

Indeed, that example is using a name that's not defined anywhere...

The "Variable Rules" section of the Quickstart would be a reasonable place to introduce the escaping functionality.  I can create a PR with a proposed change.

Created https://github.com/pallets/flask/pull/3529.
Adding the note about escaping user input in the Variable Rules section makes logical sense, but it might not catch the reader's attention when placed below the table of conversion types.  I'd welcome others' feedback.  Thanks.

authors: ThiefMaster pgkrit

repo: pallets/flask


text:
typo on useMedia example

A small typo on the useMedia example

Good catch, thanks!

authors: Julienng crosscompile

repo: palmerhq/the-platform


text:
Spark dataframe support

Is your feature request related to a problem? Please describe.
pandera is a great library, but it doesn't support pyspark DataFrame yet
Describe the solution you'd like
Schema model for pyspark DataFrame
import pyspark
import pandera as pa
from pandera.typing.pyspark import DataFrame, Series

class Schema(pa.SchemaModel):
    foo: Series[str]

spark_df = spark.read.parquet('foo.parquet')
spark_df = Schema(spark_df)

pip install pandera[pyspark]

hey @Niccolum pandera now supports pyspark.pandas dataframes! https://pandera.readthedocs.io/en/stable/pyspark.html
Out of curiosity: I haven't tested validating OG pyspark dataframes, I suspect that is not supported... is this a feature that you'd want, or is converting pyspark DFs to pyspark.pandas DFs sufficient?

authors: Niccolum cosmicBboy

repo: pandera-dev/pandera


text:
A double framebuffer for graphics

add a double framebuffer in graphics.cpp and i made a litle animation test in same file.

Closes #257

Overall pretty good! Just make sure you double check the code for common syntax mistakes before submitting. If you use VSCode for your IDE, you can install the clang-format plugin and then right click the code and click Format Document to automatically format using our style guide.

I missed some mistakes, my bad.
I use vim, i'll later search a plugin for made this automatically 😅

I use vim

vi / vim for the win!

authors: Jmallone the-grue

repo: panix-os/panix


text:
worker exits from a panic: reflect: call of reflect.Value.Len on ptr Value

worker exits from a panic: reflect: call of reflect.Value.Len on ptr Value

Please add more details like example code.


text:
feat: authenticate API requests with GITHUB_TOKEN to avoid rate-limit

GH limits unauthenticated API requests to 60 an hour - GHA can authenticate using github.token which is always present to raise the limit to 1000 per hour.
Note that the GH API is only used for determining the latest version of Terminus so anyone specifying a specific version will not be rate limited, since the release download endpoint is not subject to rate limiting.

How are the 60/hour requests counted when the requests originate from GHA?  Is it per action, per project?

GHA can authenticate using github.token which is always present
Okay, please forgive my ignorance, but just to be clear, all GH actions already have github.token defined without any configuration steps being necessary, including at the project level?

Tracking internally as FEAT-1082

You're the original maintainer.  I trust your judgment.

@G-Rath should I tag a release?  1.0.1?  1.1?

How are the 60/hour requests counted when the requests originate from GHA? Is it per action, per project?

Honestly I'm not entirely sure beyond knowing that is it not less than "per workflow" - it could be per project, but the docs only say "Unauthenticated requests are associated with the originating IP address". When using the GITHUB_TOKEN through:

When using GITHUB_TOKEN, the rate limit is 1,000 requests per hour per repository.

just to be clear, all GH actions already have github.token defined without any configuration steps being necessary, including at the project level?

That is my understanding yes - according to these docs:

An action can access the GITHUB_TOKEN through the github.token context even if the workflow does not explicitly pass the GITHUB_TOKEN to the action

I've had a hunt around and not been able to find anyway to definitely completely disable the GITHUB_TOKEN (/ github.token), nor a clear way to remove permission for that particular API.
We've already been using this on another action I maintain without issue, and if someone does report an issue because of this change I'm happy to be pinged and look into it (especially since it'll probably impact my other action 😅)

should I tag a release? 1.0.1? 1.1?

yup, I think releasing this as v1.1.0 would make sense - thanks!

When I try to draft a new release, it shows this message saying it won't publish to GH marketplace because the action name isn't unique?  I searched GH marketplace for "Setup Pantheon Terminus" and this action was the only result...  :/

authors: G-Rath namespacebrian


text:
New Rules: CS EventStream Audit Events

Background
The Crowdstrike Eventstreams log source includes several audit events which provide security value. These detections provide some baseline coverage against these logs.
Changes

multiple new detections, covering user management and IP allowlist activity

Testing

Added multiple unit tests for each detection

Would love some eyes on the IP allowlisting rules. I'm concerned they may be too noisy!

repo: panther-labs/panther-analysis


text:
Expand the range of supported setuptools.

We now just exclude the island where setuptools ran a disruptive
de-vendoring experiment.

I manually tested pex --pre with setuptools versions inside and outside the excluded range, only getting failures in the excluded range. I used https://github.com/pypa/setuptools/blob/master/CHANGES.rst to find the exclusion range in the first place.
The motivation here is to let Pants use a newer version of setuptools so that it can run under python 3.7. See: https://github.com/pantsbuild/pants/issues/6363

repo: pantsbuild/pex


text:
AttributeError: 'list' object has no attribute 'hbp'

 hbp_home = player_stats.home_batting.hbp
AttributeError: 'list' object has no attribute 'hbp'

^ Getting this error on the player stats object
    for game in games:
        stats = mlbgame.team_stats(game.game_id)
        player_stats = mlbgame.player_stats(game.game_id)
        hbp_home = player_stats.home_batting.hbp

Also is there a way to grab or calculate advanced sabermetrics like wRC, FIP, wOPA, etc for teams(not players) on a game level basis. I am working on some ML models and that would be really helpful. Would do any API work myself just want to know if it can be done.

MLB doesn't provide this info as a team stat. You will have to calculate it per player:
player_stats = mlbgame.player_stats(game.game_id)
hbp_home = sum([ p.hbp for p in player_stats.home_batting ])

copy that! and

Also is there a way to grab or calculate advanced sabermetrics like wRC, FIP, wOPA, etc for teams(not players) on a game level basis. I am working on some ML models and that would be really helpful. Would do any API work myself just want to know if it can be done.

Sabremetrics would have to be calculated based on individual statistics and then summed. Fangraphs has some information on how to calculate stats like wRC, FIP, and wOPA. You should be able to build arrays and then build list comprehension summations from those individual stats.

@trevor-viljoen has provided the correct answer, closing.

authors: Hisairnessag3 panzarino trevor-viljoen

repo: panzarino/mlbgame

license: mit


text:
Update game.py

The XML pulled by data.get_scoreboard doesn't have the probable pitchers in the same order as the teams.  Teams are in positions Home = [0] Away = [1] , while the Probable Pitchers are in positions Home = [1] Away = [0]  .  I have gone to make the change on lines 108 and 115 of game.py

I realize I'm late to the game here, but this "fix" might have caused the home and away probably pitchers to be flipped. According to ESPN the probably starters for 6/29/21 Rays at Nationals is Rich Hill (TB) and Joe Ross (WSH). But if I execute this code:
games = mlbgame.day(2021,6,29) a_game = games[0] print(a_game.away_team) print(a_game.p_pitcher_away) print() print(a_game.home_team) print(a_game.p_pitcher_home)
I get this output:
`Rays
Joe Ross
Nationals
Rich Hill`

authors: baxterdax daveklotz


text:
Add support for PayMaya

Adds support for PayMaya and adds a new type called AllowedPaymentMethods.

Thank you so much for the contribution @pdlozano! I'll review this when as soon as I have the time. Have a great day :)

authors: paolotiu pdlozano

repo: paolotiu/paymongo


text:
Fix compose file

I had to edit the compose file to fix an issue of Traefik not allowing me to access the webpage from another machine.
I would get a 404 Page Not Found error uintill i did the fix below.

Replace all instances of papermerge.local with ${HOSTNAME} in docker-compose.yml
add HOSTNAME=<hostname of PC> to .env

You know, I just had the thought to instead of make it a .env value, to get the hostname from the PC itself. That might be easier.

@gljones2001 I would gladly accept your pull request with above mentioned fix :smile:

oh. I didn't have a pull request. I just manually fixed it by using the web editor in Portainer.

Honestly it was such a simple edit I thought you could just do it through the github web editor. I didn't want to do a whole pull request just for like 2 lines of code.

There are the places I changed it.

Awesome dude. Thanks. I'll try it out.

Hey, you might want to update the setup page in the docs.
https://docs.papermerge.io/Installation/docker.html

Also, Noticed a descrepicy in the .env on the readme.
APP_IMAGE=ghcr.io/papermerge/papermerge
APP_TAG=latest
PAPERMERGE_JS_IMAGE=ghcr.io/papermerge/papermerge.js
PAPERMERGE_JS_TAG=latest

HOSTNAME=docman

DB_USER=postgres
DB_NAME=postgres
DB_PASSWORD=postgres
DB_HOST=db
DB_PORT=5432

REDIS_HOST=redis
REDIS_PORT=6379

ES_HOSTS=es
ES_PORT=9200

SECRET_KEY=alsdkalsdjlaksdj90823423!KLKJLkjkjlkjlKLPOgrwqna

SUPERUSER_USERNAME=admin
SUPERUSER_EMAIL=admin@mail.com
SUPERUSER_PASSWORD=password

Notice the ghcr.io/ that is not on the readme.

good catch

repo: papermerge/papermerge-core


text:
bump duckdb-rs version

Ticket(s) Closed

Closes #153 -

What
support reading and writing geoparquet
Why
with duckdb_spatial>=v1.1.0 geometry column in parquet will be read/write as geometry instead of blob
How
no explicit change of sql is needed
Tests
a read/write parquet test should be added later

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Deo seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

This PR would need documentation so people know how to use it

This PR would need documentation so people know how to use it

just fix the packages and now i can build it and test it locally
it seems there's no geometry type cell.rs, currently it still read the geometry column as text :(

This PR would need documentation so people know how to use it

just fix the packages and now i can build it and test it locally
it seems there's no geometry type cell.rs, currently it still read the geometry column as text :(

Yes, that’s because duckdb-rs is on duckdb 1.0.0, not 1.1.0 which it seems like it needs. We would need you to submit a PR to our fork to duckdb-rs to upgrade it as well there, then update the commit here.

This PR would need documentation so people know how to use it

just fix the packages and now i can build it and test it locally
it seems there's no geometry type cell.rs, currently it still read the geometry column as text :(

Yes, that’s because duckdb-rs is on duckdb 1.0.0, not 1.1.0 which it seems like it needs. We would need you to submit a PR to our fork to duckdb-rs to upgrade it as well there, then update the commit here.

from https://github.com/paradedb/duckdb-rs/tree/main/crates/libduckdb-sys it seems it's already on v1.1.1

This PR would need documentation so people know how to use it

just fix the packages and now i can build it and test it locally
it seems there's no geometry type cell.rs, currently it still read the geometry column as text :(

Yes, that’s because duckdb-rs is on duckdb 1.0.0, not 1.1.0 which it seems like it needs. We would need you to submit a PR to our fork to duckdb-rs to upgrade it as well there, then update the commit here.

from https://github.com/paradedb/duckdb-rs/tree/main/crates/libduckdb-sys it seems it's already on v1.1.1

Ah yes! You are right! Oops, I had no idea. Then, I'm not sure... Are you sure this is supported? Do we need to do anything to enable it?

Could you please sign the CLA and I think this will be good to go :)

done :)
for the geoparquet reference, i think it need some extra data type mapping (especially postgis geometry)
https://github.com/duckdb/duckdb/pull/12503
https://github.com/duckdb/pg_duckdb/issues/141

Could you please sign the CLA and I think this will be good to go :)

done :)
for the geoparquet reference, i think it need some extra data type mapping (especially postgis geometry)
duckdb/duckdb#12503 duckdb/pg_duckdb#141

We ship with Postgis support and the spatial extension is enabled in pg_analytics. It was added a while back. Does that give you enough?

Could you please sign the CLA and I think this will be good to go :)

done :)
for the geoparquet reference, i think it need some extra data type mapping (especially postgis geometry)
duckdb/duckdb#12503 duckdb/pg_duckdb#141

We ship with Postgis support and the spatial extension is enabled in pg_analytics. It was added a while back. Does that give you enough?

not enough at the moment.
we want to store the analytic data into s3 in parquet format, most of them are gis data, with newly duckdb, we could dump them as geoparquet, and read it back directly as geometry data type
load spatial;
create table gis_test as select st_makepoint(127.0, 30.0) as geometry;
copy  gis_test to '~/Downloads/gis_test.parquet' (format parquet);
create table gis2 as select * from '~/Downloads/gis_test.parquet';
describe table gis2;
┌─────────────┬─────────────┬─────────┬─────────┬─────────┬─────────┐
│ column_name │ column_type │  null   │   key   │ default │  extra  │
│   varchar   │   varchar   │ varchar │ varchar │ varchar │ varchar │
├─────────────┼─────────────┼─────────┼─────────┼─────────┼─────────┤
│ geometry    │ GEOMETRY    │ YES     │         │         │         │
└─────────────┴─────────────┴─────────┴─────────┴─────────┴─────────┘

but in pg_analytics, it still read it as text column
CREATE FOREIGN TABLE gis2 () server parquet_server options(files '~/Downloads/gis_test.parquet');
select * from information_schema.columns where table_schema = 'public' and table_name = 'gis2' and column_name = 'geometry';
select column_name, data_type from information_schema.columns where table_schema = 'public' and table_name = 'gis2' and column_name = 'geometry';
 column_name | data_type
-------------+-----------
 geometry    | text
(1 row)

Could you please sign the CLA and I think this will be good to go :)

done :)
for the geoparquet reference, i think it need some extra data type mapping (especially postgis geometry)
duckdb/duckdb#12503 duckdb/pg_duckdb#141

We ship with Postgis support and the spatial extension is enabled in pg_analytics. It was added a while back. Does that give you enough?

not enough at the moment. we want to store the analytic data into s3 in parquet format, most of them are gis data, with newly duckdb, we could dump them as geoparquet, and read it back directly as geometry data type
load spatial;
create table gis_test as select st_makepoint(127.0, 30.0) as geometry;
copy  gis_test to '~/Downloads/gis_test.parquet' (format parquet);
create table gis2 as select * from '~/Downloads/gis_test.parquet';
describe table gis2;
┌─────────────┬─────────────┬─────────┬─────────┬─────────┬─────────┐
│ column_name │ column_type │  null   │   key   │ default │  extra  │
│   varchar   │   varchar   │ varchar │ varchar │ varchar │ varchar │
├─────────────┼─────────────┼─────────┼─────────┼─────────┼─────────┤
│ geometry    │ GEOMETRY    │ YES     │         │         │         │
└─────────────┴─────────────┴─────────┴─────────┴─────────┴─────────┘

but in pg_analytics, it still read it as text column
CREATE FOREIGN TABLE gis2 () server parquet_server options(files '~/Downloads/gis_test.parquet');
select * from information_schema.columns where table_schema = 'public' and table_name = 'gis2' and column_name = 'geometry';
select column_name, data_type from information_schema.columns where table_schema = 'public' and table_name = 'gis2' and column_name = 'geometry';
 column_name | data_type
-------------+-----------
 geometry    | text
(1 row)

Can you confirm the duckDB version you are using separately? And can you confirm the GIS extension is loaded in pg_analytics?

confirm duckdb version 1.1.1 and 1.1.3 works with geoparquet
confirm postgis is loaded in postgres
but I'm not sure how to check if duckdb spatial is enabled  in pg_analytics :(
tried select duckdb_execute($$load spatial$$); with no luck

authors: CLAassistant DeoLeung philippemnoel

license: PostgreSQL


text:
chore: improve channel is full log

the log wasn't accurate because we're keeping full channels around

Codecov Report

Merging #2595 (0f464f8) into main (bc5e150) will decrease coverage by 57.43%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #2595       +/-   ##
===========================================
- Coverage   71.47%   14.05%   -57.43%
===========================================
  Files         493      493
  Lines       61924    61924
===========================================
- Hits        44262     8704    -35558
- Misses      17662    53220    +35558

Flag
Coverage Δ

integration-tests
14.05% <0.00%> (-4.03%)
:arrow_down:

unit-tests
?

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

Impacted Files
Coverage Δ

crates/transaction-pool/src/pool/mod.rs
6.18% <0.00%> (-42.91%)
:arrow_down:

... and 375 files with indirect coverage changes

authors: codecov-commenter mattsse

repo: paradigmxyz/reth


text:
chore(github): use codespell to inspect and correct spelling issues

use https://github.com/codespell-project/codespell to lint the spelling issues.

I like this in general, but think we have a policy against mining, no @rkrasiuk ?

sorry, I keep finding new things. some questions:

how do I run this codespell locally?
can we add it to the makefile? we have a make pr (or something similar)

sorry, I keep finding new things. some questions:

how do I run this codespell locally?
can we add it to the makefile? we have a make pr (or something similar)

thanks for the advice, add that into makefile and as as a sep task of lint, now we can run make lint-codespell to lint it.

authors: emhane jsvisa onbjerg


text:
RAM checksum error on COMxx (P1 Flip)

I just installed the 2.9.0 version of the Propeller-Tool onto my Windows 7 laptop.
For some reason I can't successfully load into RAM without getting a checksum error.
I've tried two different P1 Flip modules, which in turn use two different COM ports 31 and 39.
If I try using the 2.8.0 version of Propeller-Tool with the same P1 Flip modules and the same COM ports, everything loads fine.
If I load a P2 program into a P2 Edge with the 2.9.0 version, everything loads fine.
I've moved the original Propeller-Tool folder out of the way and re-installed 2.9.0 and I still can't get either P1 Flip to load successfully.
So what is causing the issue with 2.9.0 when working with P1 Flip modules?

Thank you for reporting.  I verified the problem; it happens on my system too.  It appears a change I made caused this problem and I failed to download to a P1 afterwards.  I'm looking into it now.  There were a number of structural changes I had to make to support one or more of the recent changes, including to how the compilation process works.  It's probably related to that, though is certainly a surprise to me.

Found it and fixed it!  Solved in v2.9.1.

authors: PropGit francisbauer

repo: parallaxinc/Propeller-Tool


text:
Add demo reload function

Adding Spin code within the block definitions is unworkable because something in the Blockly library is adding spaces to the beginning of the lines. Moving the new code into the Blockly_Wrapper.spin file since that is where everything else seems to live.

Perfect @zfi - I wish I had thought of and suggested that earlier.

authors: PropGit zfi


text:
Revert "Remove stealth link from all newsletters"

Reverts parameter1/science-medicine-group-newsletters#74

 Remove the hidden links in templates - all sites


text:
#253: created an IconButton component in papi-components

This change is

lib/papi-components/src/icon-button.component.tsx line 32 at r1 (raw file):
Previously, tjcouch-sil (TJ Couch) wrote…
Generally we try to go for fewer props so we aren't committing to too many props in case we change in the future. I'd recommend removing isFocusRippleDisabled and adjustMarginToAlignToEdge. Not a huge deal right now either way.

I did wonder about isFocusRippleDisabled. From what I read, I think that the second one is likely to be needed, as there doesn't seem to be any other reasonable way to achieve proper alignment with other controls inside a parent container.

repo: paranext/paranext-core


text:
MNT add dispatcher parameters inside the config file

We can move the parameter of the dispatcher inside the event config file since we are triggering a dispatcher by event. It will be much simpler to launch them then.

what I would expect as behavior is that we have default parameters for dispatcher for each event (eg 2 workers and 2 threads each) but that config can allow you to overwrite these values. is that we is done here?

Yes apart that the default of worker that can be started started is the number of CPU and we don't limit the number of openmp threads. It was the previous default but we could change to a more sensible default (at least for the workers)

authors: agramfort glemaitre

repo: paris-saclay-cds/ramp-board


text:
Impl Codec for Compact and Compact

Should be useful for https://github.com/paritytech/substrate/issues/847

Could you please rebase with master, so that we can test that CI works?

authors: bkchr pepyakin

repo: paritytech/parity-codec


text:
Standard way to initialize a pallet after genesis

in the future probably would be nice to have an initialize function coming from the pallet itself.
Originally posted by @shawntabrizi in https://github.com/paritytech/polkadot/pull/5582#discussion_r942502407
Currently, we can easily initiate a pallet in the runtime by using on_runtime_upgrade. This is fine, but we can dp better and create a standard process for this.
Also, in the academy, we realized that the students are quite confused about this and mistakenly think that genesis_build is called upon creation of the pallet, not creation of the whole runtime.

duplicate of https://github.com/paritytech/substrate/issues/8373

repo: paritytech/substrate


text:
Block fullness targeting with fee multiplier

Detailed research here
Motivation

We want to target block saturation level of s* = 0.25 (adjustable). The allows the system to handle 4x transaction spikes, with room for crucial transactions, e.g. misbehavior reports.
We introduce v a fee variability (multiplier) factor, which controls how quickly the transaction fees adjust. Where: v≤ p / (k * (1−s∗)).

Fast vs Slow fee mechanisms
There are two options to consider when it comes to the fee adjustment rate

Slow adjusting with tipping (recommended): Adjusts slowly, at the pace of long term tendencies, and uses tipping to deal with long queues at peak hours. V is small.

Fast adjusting: Adjusts the price very quickly, at the same pace as the peaks and valleys of activity.

from: @gavofyork

So one fundamental, non-changing part of this task is to communicate some sense of weight from each transaction to a centralized store: probably executive module or write system module directly.
Ideally, I think this weight should be some enum like
pub enum Weight {
    /// Simple tx, irrelavent of the inputs
    Constant,
    /// Weighted
    Weighted(u32),
    /// Red flag.
    Maximum,
     /// Transaction is actually helpful and should be rewarded (indices module etc.)
     ShouldReward(u32)
    ....
}

But for now, we can assume we just need to communicate one u32 value back.
I see a few options to do that:

Return it via the default Ok(). I don't know where it goes exactly atm but I expect it to be easily capturable.
pass a secondary, optional argument into each transaction (after origin) which has the type Fn(w: Weight) -> (). Essentially pas a setter for weight into each tx.  If not used, then Default implementation of Weight will be used.
Similar to the above but with a global function which will be reserved keyword like set_weight(w).

If it is possible, I returning Ok(something) seems quite simple and gets the job done.
Any initial feedback? @bkchr @gavofyork @shawntabrizi

@Kianenigma we had a primitive implementation of this centralized transaction fee/weight store. I wonder if you guys can take a look to see if it helps in this case? The ultimate goal is same that use transaction fee to control the target block fullness. As the first step we developed a fees registry (based on the srml-fees module I developed which got removed from substrate).
You can take a code at https://github.com/cennznet/plug-blockchain (our fork of Substrate).
A few more traits
https://github.com/cennznet/plug-blockchain/blob/67ed189597ecbd50cf080fdf3584059fcbdc98fc/srml/support/src/additional_traits.rs
Fees module and decl_fee to declare a fee enum
https://github.com/cennznet/plug-blockchain/blob/67ed189597ecbd50cf080fdf3584059fcbdc98fc/prml/fees/src/lib.rs#L46-L53
Another usage of decl_fee
https://github.com/cennznet/plug-blockchain/blob/67ed189597ecbd50cf080fdf3584059fcbdc98fc/prml/generic-asset/src/lib.rs#L213-L218
Implementation of the traits, which needs refactor so the calculation part are in each individual module instead of a single file.
https://github.com/cennznet/plug-blockchain/blob/67ed189597ecbd50cf080fdf3584059fcbdc98fc/srml/support/src/additional_traits.rs
Our implementation definitely needs a lot of improvements, but I believe it is a good start.

I'm reading the doc:

A transaction fee includes a base fee and a variable fee that is proportional to the number of bytes to be put on chain. That is,
f(tx):=fbase+size(tx)⋅fbyte,
for some parameters fbaseand fbyte and where size(tx) is the number of bytes of transaction. The base fee will depend on the type of transaction, while the per-byte fee may be the same for all types.

what is a type here ? should contract_put_code_extrinsic be its own type for instance ?
considering this statement it seems, maybe we should allow modules to defines there own types (like we do for origin) and they must specify for each call a type. (with an attribute maybe or somewhere else)

The derived formula for v seems to be valid, but it is a very loose bound. From here, we have
\frac{f_{final byte} - f_{initial byte}}{f_{initial byte}} ~= vk(s_avg - s*) \leq vk(1-s*) \leq p

This makes sense because s_avg \leq 1, but we use it to derive the bound v \leq \frac{p}{k(1-s*)} and while that is correct, it is not as strict a bound as if we replace 1 with s_avg => our new, more strict bound is v \leq \frac{p}{k(s_avg - s*)}.
My only concern is that now v can be negative; specifically v is negative if the average saturation is below the target saturation. This still doesn't mean the fee multiplier itself is ever negative. For clarification, the actual fee multiplier is NOT v, it is
1 + v(s-s*) + v^{2}(s-s*)^{2}/2

where s* is the target saturation and s is the current block's saturation

closed via #2854

authors: 4meta5 Kianenigma nczhu thiolliere xlc


text:
Referendum decision period quantization

Right now if AlarmInterval set to > 1, the time for alarm will be calculated to the past, and scheduler will fail.
This PR fixes it.

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2103277

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2103276

Please add a test.

Please see https://github.com/paritytech/substrate/pull/12815 and check if it passes your test.

Please see #12815 and check if it passes your test.

got it wrong actually in this PR, please check an addition to your pr, test and note_one_fewer_deciding uses set_alarm  https://github.com/paritytech/substrate/pull/12818

Closing in favor of this pr https://github.com/paritytech/substrate/pull/12815

authors: gavofyork muharem paritytech-cicd-pr


text:
Some code cleanups

What this PR does / why we need it:
Some code cleanup.
Change type

[ ] New feature
[ ] Bug fix
[ ] Unit tests
[ ] Integration tests
[ ] CI
[ ] Documentation
[ ] Auto generated SDK code

Impacted services

[ ] Workflow Service
[ ] Notifivcation Service

Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.

/lgtm
/approve

authors: masayag nirarg

repo: parodos-dev/parodos


text:
Update CI to merge automated PR if one already exists for the workflow

Closes https://issues.redhat.com/browse/FLPATH-1698
Update CI to merge automated PR is one already exists for the workflow

pls resolve conflicts.

authors: gabriel-farache masayag

repo: parodos-dev/serverless-workflows


text:
colors.py might be missing I previous merge

@tlapusan please see: https://github.com/parrt/dtreeviz/pull/185/files#diff-c79359e805463205f1a81b864ceee524b59f173b5dbd88111f2a9b256f9d65f2R32

what should be the bug ?

Sorry for the lack of clarity. It looks like that function is missing if you follow that link but I could be confused.

authors: parrt tlapusan

repo: parrt/dtreeviz


text:
Is Facebook Login Implemented?

I was taking a look on how to implement Facebook Login with Parse and found out that ParseFacebookUtils is not presented in the project and found no implementation what so ever of it. Is it implemented?

No, the facebook utils are no longer part of the master branch.
If you want to check them out you would need to dig in past commits (e.g. those made for version 1.7 or earlier).

In my case, i’ve already implemented the Facebook login with the Xamarin.Auth package, after I sign up with the Facebook account, how do I link it with a new parse user? Because when i create a new parse user i need to provide a password as well, in the facebook case it is not needed. My question is the same for when I log in with Facebook, how do I log in with Facebook without the ParseUser password

And in this 2.0 pre-release, are Push Notifications now implemented?

I will address the lack of ParseFacebookUtils soon.

@TobiasPott Where did the 1.8.0 branch go? I needed that for reference. Release 1.7.0 contains the source for that release obviously, but the 1.8.0 branch had some extra changes in it that the previous maintainers were working on. Would prefer to have that in a "old-codebase" branch or something if you still have access to a copy. The source I currently have access to will do fine for now though. Will implement a login utilities file soon as well as live query.

And in this 2.0 pre-release, are Push Notifications now implemented?

The infrastructure does exist but I have no idea if it works or is usable; will have to test.

Just to be clear, any OS-or-platform-specific code is not present in the current codebase. Will have to add this back in some form later.

@TheFanatr I'll try to give it a go during this week and check how to get access to the facebook SDK and an environment for testing it.

@TheFanatr I've pushed the 1.8.0 legacy codebase to https://github.com/parse-community/Parse-SDK-dotNET/tree/legacy-codebase/1.8.0
I've also added 1.7.0 as a separate branch according to its tag in the history (I cannot resemble whether or not any later commit got into the actual version 1.7.0 release on nuget).

@TobiasPott Perfect! No worries; I was using 1.8 as a reference because it was to be a fallback for the master branch if for some reason the Parse admins decided to revert the initial 2.0 refactor, so I assumed it was the best branch to base off of. Also, I believe the old maintainers had started or continued a modularization effort in it before it went stale, so it's a better source of code to take back over into master when needed, as issues from 1.7 could have been fixed.

Could you elaborate which the latest .NET 5 version needs to be installed?

The latest preview of .NET 5 can be found on the .NET website by clicking on the link to the installer or binaries for your platform under the latest version listed.

Is it required specific for the facebook auth option or due to other changes you made to develop?

It is due to the use of C# 9 features in the added and modified code. It at least needs the latest .NET Core CLI.

Thank you for the quick response.
I understand why to choose the upcoming .NET 5 to access newer C# features and I'm torn between using new features and limiting the SDK to a lower feature level to maintain compatibility with Unity (they lack far behind regarding C# feature level or the .NET version).
I would like to discuss as we would reduce the use of the .NET SDK with latest .NET and C# features to standard .NET applications or Xamarin and kick out any use in Unity unless they add support for .NET 5.
I couldn't find any specific roadmap from Unity on this but they point out that they only start working on it after the official release (https://forum.unity.com/threads/net-5-support.839890/) unfortunately.

Well, all official .NET platforms will support .NET 5 including Xamarin, but so long as the SDK is kept at .NET Standard 2.0, which it still is, any compiled binary will still be compatible with Unity. The .NET 5 SDK is only needed for builds as it bundles the latest version of the C# compilation tooling.

I want to upgrade to .NET Standard 2.1 so I can use System.Text.Json and Span<T> without needing to reference the compatibility shim packages for them. This would allow for faster processing, and potentially using a fast Span<T>-based web socket library if an applicable one exists. What I might do is reluctantly fall into a similar trap the previous maintainers fell into with making slightly differing versions to support different target frameworks. To be clear these wouldn't be separate versions completely, just would have better implementations of things where available, and I think it would be more acceptable than before because it's not for every platform, it's just effectively the default (.NET X) implementation, then the Unity one, where those two would only be different if some improvement is made possible by .NET X but is unavailable on Unity's latest supported .NET target.

Ah yes, thank you for the clarification I overread/misinterpreted it not changing the SDK to .NET 5 but only require the SDK installed for toolchain changes.
Sorry for the mishap on my side.

Hello @TobiasPott @TheFanatr Getting the error below:
Using Xamarin.Forms Standard 2.0 - Code in App.xaml.cs, called in "public App(){}"
Question: Getting "Null References Errors". Am I setting this up wrong?
`ServerConnectionData serverConnectionData = new ServerConnectionData
{
ApplicationID = Const.Parse_App_ID,
ServerURI = Const.Parse_Server,
Key = Const.DOT_NET_KEY
};
ServiceHub serviceHub = null;
// Initialize FB Parse Login
//var facebook = new FacebookAuthenticationMutator { Caller = Const.FB_App_ID };
        ParseClient parseClient = new ParseClient(serverConnectionData);
        //parseClient = new ParseClient(serverConnectionData, serviceHub, facebook);
        parseClient.Publicize();
        parseClient = ParseClient.Instance;

        //Register Installation to Parse
        ParseInstallation parseInstall = new ParseInstallation();
        parseInstall = parseClient.GetCurrentInstallation();
        parseInstall.SaveAsync().GetAwaiter();`

at Parse.Infrastructure.HostManifestData.get_Inferred () [0x00000] in /Users/adesegunadesegun/Documents/1. Work/5.4 Xamarin/External_Libraries/parse-community/Parse-SDK-dotNET/Parse/Infrastructure/HostManifestData.cs:23
at Parse.Infrastructure.ServiceHub+<>c.<get_MetadataController>b__8_0 () [0x00000] in /Users/adesegunadesegun/Documents/1. Work/5.4 Xamarin/External_Libraries/parse-community/Parse-SDK-dotNET/Parse/Infrastructure/ServiceHub.cs:40
at Parse.Infrastructure.Utilities.LateInitializer.GetValue[TData] (System.Func1[TResult] generator) [0x00067] in /Users/adesegunadesegun/Documents/1. Work/5.4 Xamarin/External_Libraries/parse-community/Parse-SDK-dotNET/Parse/Infrastructure/Utilities/LateInitializer.cs:24    at Parse.Infrastructure.ServiceHub.get_MetadataController () [0x00000] in /Users/adesegunadesegun/Documents/1. Work/5.4 Xamarin/External_Libraries/parse-community/Parse-SDK-dotNET/Parse/Infrastructure/ServiceHub.cs:40    at Parse.Abstractions.Infrastructure.CustomServiceHub.get_MetadataController () [0x00000] in /Users/adesegunadesegun/Documents/1. Work/5.4 Xamarin/External_Libraries/parse-community/Parse-SDK-dotNET/Parse/Abstractions/Infrastructure/CustomServiceHub.cs:22    at Parse.ParseInstallation.SaveAsync (System.Threading.Tasks.Task toAwait, System.Threading.CancellationToken cancellationToken) [0x00037] in /Users/adesegunadesegun/Documents/1. Work/5.4 Xamarin/External_Libraries/parse-community/Parse-SDK-dotNET/Parse/Platform/Installations/ParseInstallation.cs:198    at Parse.ParseObject+<>c__DisplayClass91_0.<SaveAsync>b__0 (System.Threading.Tasks.Task toAwait) [0x00000] in /Users/adesegunadesegun/Documents/1. Work/5.4 Xamarin/External_Libraries/parse-community/Parse-SDK-dotNET/Parse/Platform/Objects/ParseObject.cs:573    at Parse.Infrastructure.Utilities.TaskQueue.Enqueue[T] (System.Func2[T,TResult] taskStart, System.Threading.CancellationToken cancellationToken) [0x00025] in /Users/adesegunadesegun/Documents/1. Work/5.4 Xamarin/External_Libraries/parse-community/Parse-SDK-dotNET/Parse/Infrastructure/Utilities/TaskQueue.cs:62
at Parse.ParseObject.SaveAsync (System.Threading.CancellationToken cancellationToken) [0x00014] in /Users/adesegunadesegun/Documents/1. Work/5.4 Xamarin/External_Libraries/parse-community/Parse-SDK-dotNET/Parse/Platform/Objects/ParseObject.cs:573
at CheapFuel.App..ctor () [0x0006f] in /Users/adesegunadesegun/Documents/1. Work/5.4 Xamarin/CheapFuel/CheapFuel/CheapFuel/App.xaml.cs:98
at CheapFuel.iOS.AppDelegate.FinishedLaunching (UIKit.UIApplication app, Foundation.NSDictionary launchOptions) [0x0009a] in /Users/adesegunadesegun/Documents/1. Work/5.4 Xamarin/CheapFuel/CheapFuel/CheapFuel.iOS/AppDelegate.cs:86
at (wrapper managed-to-native) UIKit.UIApplication.UIApplicationMain(int,string[],intptr,intptr)
at UIKit.UIApplication.Main (System.String[] args, System.IntPtr principal, System.IntPtr delegate) [0x00005] in /Library/Frameworks/Xamarin.iOS.framework/Versions/14.2.0.12/src/Xamarin.iOS/UIKit/UIApplication.cs:86
at UIKit.UIApplication.Main (System.String[] args, System.String principalClassName, System.String delegateClassName) [0x0000e] in /Library/Frameworks/Xamarin.iOS.framework/Versions/14.2.0.12/src/Xamarin.iOS/UIKit/UIApplication.cs:65
at CheapFuel.iOS.Application.Main (System.String[] args) [0x00001] in /Users/adesegunadesegun/Documents/1. Work/5.4 Xamarin/CheapFuel/CheapFuel/CheapFuel.iOS/Main.cs:17

authors: SOGAPPS TheFanatr TobiasPott diasleo98

repo: parse-community/Parse-SDK-dotNET


text:
#60: Adding 4 missing licenses in source code.

This will resolve #60.

[Current coverage][cc-pull] is 90.94%

Merging [#65][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master        #65   diff @@
==========================================
  Files            76         76
  Lines           939        939
  Methods           0          0
  Messages          0          0
  Branches        178        178
==========================================
  Hits            854        854
  Misses           49         49
  Partials         36         36

Powered by Codecov. Last updated by [8e2e284...62dd609][cc-compare]
[cc-base-branch]: https://codecov.io/gh/parsingdata/metal/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/parsingdata/metal/compare/8e2e284b53400ed7bece2b7156bae2049187aa66...62dd6094be5b23d422103716c48856298e8f3c0d?src=pr
[cc-pull]: https://codecov.io/gh/parsingdata/metal/pull/65?src=pr

authors: codecov-io rdvdijk

repo: parsingdata/metal


text:
added topicstobecovered md file & added ppt

I have completed the initial revision of the PowerPoint presentation. However, there are a few more edits to be made. Specifically, I need to incorporate educational videos and quizzes into the presentation.

Kindly move the docx file within the project folder:

worked on comments provided

added final version of PPT for module 1

transitioned from using PowerPoint to Google Docs for presentation slides. I have included a link in the README.md file that allows you to preview the slides. Additionally, you'll find a link for the project planning related to Module 1.

transitioned from using PowerPoint to Google Docs for presentation slides. I have included a link in the README.md file that allows you to preview the slides. Additionally, you'll find a link for the project planning related to Module 1.

It still says that I need access.

repo: parthasarathydNU/humanitarians-OPT-fork


text:
Set up the Development Environment

User Story
Title: Set up the Development Environment
Description: As a developer, I want to set up a development environment with all required tools and dependencies to work on the accounts microservice.
Acceptance Criteria

The code repository is successfully cloned to the local machine.
All required dependencies are installed without any errors (e.g., programming languages, libraries, and frameworks).
The development environment includes necessary tools like a code editor (e.g., VSCode), version control (e.g., Git), and Docker (if applicable).
A script or documentation file is provided with clear setup instructions.
The accounts microservice runs locally without any issues or errors.
All team members can reproduce the setup without additional configurations or troubleshooting.

Estimated Story Points: 5


text:
feat: improved documentation of add-to-existing-project

Added manual installation steps to the doc.

Decided to merge these into one PR since there is not many changes made.

repo: partykit/partykit


text:
Sharing failed when using Windows path

When using "E:\document\temp", the valid path cannot be divided correctly

The issue will be resolved in next release.

repo: parvardegr/sharing


text:
Everything i try to get image generation to happen will not work.

Let me start by saying thanks for getting back to me as i am also busy so appreciate your response, sorry for being short with you but i was wondering if you are watching the issues or not. Sorry but it was not meant to offend. I like your package and just need some assistance in making the image generation happen.
I have the your package installed and working great. i have created jobs to process each step of the conversions but i am at a loss on how to add this to the command.

-vf fps=1 ./snapshot-%4d.png

I need to generate images every second but i dont see anything like that in your wrapper. can you please tell me how i can add it.
my conversion to mp4 is here

public function handle()
    {
        $lowBitrateFormat = (new X264('aac'))->setKiloBitrate(500);
        $name = $mediaItems[0]->name;
        $file = $mediaItems[0]->file_name;
        $clipid = $mediaItems[0]->id;
        $slug = $title = str_slug($name, "_");

        FFMpeg::fromDisk('videos')->open($clipid . '/'. $file)
            ->addFilter(function ($filters) {
                $filters->resize(new Dimension(960, 540));
            })->export()
            ->toDisk('downloadable_videos')
            ->inFormat($lowBitrateFormat)
            ->save($slug . '.mp4');

        Log::info("Job: converted at " .  Carbon::now() . " FINISHED ");
    }

I got this to almost work but its ignoring my naming convetion and falling back on the default.
any help would be greatly appreciated.
        \FFMpeg::fromDisk('videos')
                    ->open($clipid . '/'. $file)
                   // ->exportForHLS()
                    ->addFilter(function ($filters) {
                        $filters->resize(new Dimension(432, 320));
                   $filters->extractMultipleFrames(ExtractMultipleFramesFilter::FRAMERATE_EVERY_SEC, public_path('uploads/yo'), '/', 100, 2, array('-vf', 'fps=1/1', '/rename-%03d.jpg'));
                    })
                    ->export()
                    ->toDisk('downloadable_videos')
                    ->inFormat($midBitrate)
                    ->save($slug ."/". $slug . '.mp4');

One last thing i cannot get the hls filter to function it keeps error out. Do you have an example of how to implement this

I think the problem is that the extractMultipleFrames method only takes two parameters. The output name of the frames are fixed.
You could also try to add your own filter by using the addFilter() method (see documentation).

authors: laravelcoder pascalbaljet

repo: pascalbaljetmedia/laravel-ffmpeg