Thoughts on .htaccess

Hey! Thanks for maintaining this.  I have an issue when trying to use this for local development and thought you might have some insight.  For an existing site, I have a s9y directory on my local disk in ~/mysite, with the expected directories that have custom 'stuff' in them -- ~/mysite/{uploads, templates, plugins}.  I have a compose file like this:
version: '2'

services:
  db:
    image: mysql:5.7
    ports:
      - "33306:3306"
    volumes:
      - db_data:/var/lib/mysql
    restart: always
    environment:
      MYSQL_ROOT_PASSWORD: password123
      MYSQL_DATABASE: s9y
      MYSQL_USER: s9y
      MYSQL_PASSWORD: s9y
    networks:
      - s9ynet

  s9y:
    depends_on:
      - db
    image: volkerwiegand/serendipity:latest
    ports:
      - "8000:80"
    volumes:
      - ${PWD}/docker/serendipity_config_local.inc.php:/var/www/html/serendipity_config_local.inc.php
      - ${PWD}/.htaccess:/var/www/html/.htaccess
      - ${PWD}/templates:/var/www/html/templates
      - ${PWD}/uploads:/var/www/html/uploads
      - ${PWD}/plugins:/var/www/html/plugins
    restart: always
    environment:
      S9Y_DB_HOST: db:3306
      S9Y_DB_PASSWORD: s9y
      S9Y_DB_PASSWORD: s9y
      SERVER_NAME: mysite.local:8000
      SERVER_ADMIN<EMAIL_ADDRESS>      LOG_LEVEL: info
    networks:
      - s9ynet
networks:
  s9ynet:
volumes:
    db_data:

This way, I can run things locally or inside docker.  The problem is with the .htaccess volume which has serendipity controlled configuration in it like:
# BEGIN s9y
ErrorDocument 404 /index.php
DirectoryIndex /index.php
php_value session.use_trans_sid 0
php_value register_globals off
Options -MultiViews

RewriteEngine On
RewriteBase /
RewriteRule ^serendipity_admin.php serendipity_admin.php [NC,L,QSA]
RewriteRule ^((archives/([0-9]+)\-[0-9a-z\.\_!;,\+\-\%]+\.html)/?) index.php?/$1 [NC,L,QSA]
RewriteRule ^(authors/([0-9]+)\-[0-9a-z\.\_!;,\+\-\%]+) index.php?/$1 [NC,L,QSA]
RewriteRule ^(feeds/categories/([0-9;]+)\-[0-9a-z\.\_!;,\+\-\%]+\.rss) index.php?/$1 [NC,L,QSA]
RewriteRule ^(feeds/authors/([0-9]+)\-[0-9a-z\.\_!;,\+\-\%]+\.rss) index.php?/$1 [NC,L,QSA]
RewriteRule ^(categories/([0-9;]+)\-[0-9a-z\.\_!;,\+\-\%]+) index.php?/$1 [NC,L,QSA]
RewriteRule ^archives([/A-Za-z0-9]+)\.html index.php?url=/archives/$1.html [NC,L,QSA]
RewriteRule ^([0-9]+)[_\-][0-9a-z_\-]*\.html index.php?url=$1-article.html [L,NC,QSA]
RewriteRule ^feeds/(.*) index.php?url=/feeds/$1 [L,QSA]
RewriteRule ^unsubscribe/(.*)/([0-9]+) index.php?url=/unsubscribe/$1/$2 [L,QSA]
RewriteRule ^approve/(.*)/(.*)/([0-9]+) index.php?url=approve/$1/$2/$3 [L,QSA]
RewriteRule ^delete/(.*)/(.*)/([0-9]+) index.php?url=delete/$1/$2/$3 [L,QSA]
RewriteRule ^(admin|entries)(/.+)?$ index.php?url=admin/ [L,QSA]
RewriteRule ^archive/? index.php?url=/archive [L,QSA]
RewriteRule ^(index|atom[0-9]*|rss|b2rss|b2rdf).(rss|rdf|rss2|xml) rss.php?file=$1&ext=$2
RewriteRule ^(plugin|plugin)/(.*) index.php?url=$1/$2 [L,QSA]
RewriteRule ^search/(.*) index.php?url=/search/$1 [L,QSA]
RewriteRule ^comments/(.*) index.php?url=/comments/$1 [L,QSA]
RewriteRule ^(serendipity\.css|serendipity_admin\.css)$ index.php?url=/$1 [L,QSA]
RewriteRule ^index\.(html?|php.+) index.php?url=index.html [L,QSA]
RewriteRule ^htmlarea/(.*) htmlarea/$1 [L,QSA]
#RewriteCond %{REQUEST_URI} !-U
RewriteRule (.*\.html?) index.php?url=/$1 [L,QSA]

<Files *.tpl.php>
    deny from all
</Files>

<Files *.tpl>
    deny from all
</Files>

<Files *.sql>
    deny from all
</Files>

<Files *.inc.php>
    deny from all
</Files>

<Files *.db>
    deny from all
</Files>

# END s9y

but then gets overwritten by the entrypoint.sh script:
cat >/var/www/html/$s9yUri/.htaccess <<-EOF
	php_value memory_limit ${memLimit:-128}M
	php_value upload_max_filesize ${maxSize}M
	php_value post_max_size ${maxSize}M
EOF

I can think of a few ways to work around and/or solve this issue, but they all involve wrapper scripts or changes to the docker image.  Do you have any thoughts?

Sorry, no - I'm only using the Docker version and don't change the .htaccess file. I'm afraid I can't help.


Fuse Logger API

Expose a Fuse Logger API in all environments
iOS
Will broadcast native log output to JS Console, observed by FuseRuntime
Android
Will broadcast native log output to JS Console, observed by FuseRuntime
JS
Will broadcast JS log output to the native counterpart.
This way log output is observable both while working in within the native IDEs, or inside the web inspector.
FuseRuntime shall expose a flag to determine if the app is in debug mode. Log broadcasting to save on resources should be disabled while in release mode.

Current state:
Logging is bridged from webview to iOS when using FuseLogger API. Android is still missing implementation.
Both native doesn't have a sister FuseLogger yet to bridge native logs to the webview.


[NL] Add fixed domain after name

Added fixed domain expansion rules in the domain specific files to make sure the right intent is used in case the domain word is added to the intent, but not added in the name of the device.

I initially added tests without a space.
So <name> <light> was tested both with eg werkbanklicht and werkbank licht
But I expect this to fail after https://github.com/home-assistant/intents/pull/830 is merged, so I will try to implement that after that one is merged.

#830 has been merged :+1:


add generation of multi module projects

add generation of multi module projects using Thomas Broyers project layout.
Create a new field inside the Application Meta Data card to choose between  normal maven project and multi module maven project.
Add generation of sources for multi module layout.

Feature is added.
also added:

generate a login screen and filter
generate a error screen
using FreeMarker templates for non Java files


Upsert HPO

HPO releases new files: modify the script to add/edit existing records instead of assuming an empty table.
At the moment we can drop from everywhere and re-import, but if we create keys with foreign references that wouldn't work anymore.

Is this for when we re-import the HPO ?  The HPO standard shoudl handle seconday accessions created by updates in the HPO.


Switch docs

Adding docs for missing prop

Failed pictures (1): failed-pictures.zip

:postbox: Posted by artipost.io, through Travis build 424799865, for commit 12e6960.


Revert "Spatial: Link to docs from landing page"

This reverts commit acbc1188497d65b44b0a077190828e84018f103e.
Demo was not functional due to wrong extensions compilation, should be working now.

Thanks!


Updating SAML logins to include a socketId

Proposed changes
Updating SAML logins to include a socketId
Types of changes

New feature (non-breaking change which adds functionality)

Linked Issues
https://github.com/rancherlabs/rancher-security/issues/444

No longer waiting on backend changes.


Update to CAPI  1.6

What this PR does / why we need it:
Update to CAPI  1.6
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

unit test
?   	github.com/oracle/cluster-api-provider-oci	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/config	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/metrics	[no test files]
ok  	github.com/oracle/cluster-api-provider-oci/api/v1beta1	50.385s	coverage: 23.7% of statements
ok  	github.com/oracle/cluster-api-provider-oci/api/v1beta2	0.045s	coverage: 15.6% of statements
ok  	github.com/oracle/cluster-api-provider-oci/cloud/ociutil	0.008s	coverage: 19.6% of statements
?   	github.com/oracle/cluster-api-provider-oci/cloud/scope/mocks	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/base	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/base/mock_base	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/compute	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/compute/mock_compute	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/computemanagement	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/computemanagement/mock_computemanagement	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/containerengine	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/containerengine/mock_containerengine	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/identity	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/identity/mock_identity	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/loadbalancer	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/loadbalancer/mock_lb	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/networkloadbalancer	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/networkloadbalancer/mock_nlb	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/vcn	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/cloud/services/vcn/mock_vcn	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/feature	[no test files]
?   	github.com/oracle/cluster-api-provider-oci/version	[no test files]
ok  	github.com/oracle/cluster-api-provider-oci/cloud/scope	230.098s	coverage: 75.1% of statements
ok  	github.com/oracle/cluster-api-provider-oci/cloud/util	0.086s	coverage: 61.1% of statements
ok  	github.com/oracle/cluster-api-provider-oci/controllers	26.767s	coverage: 58.7% of statements
ok  	github.com/oracle/cluster-api-provider-oci/exp/api/v1beta1	13.492s	coverage: 15.3% of statements
ok  	github.com/oracle/cluster-api-provider-oci/exp/api/v1beta2	0.017s	coverage: 4.8% of statements
ok  	github.com/oracle/cluster-api-provider-oci/exp/controllers	0.550s	coverage: 56.3% of statements
go tool cover -func=coverage.out -o coverage.txt
go tool cover -html=coverage.out -o coverage.html


Adapt to recent changes in esp-idf API

This fixes #1 for me. The issue was a upstream modification of the ledc_timer_config_t struct.
I am just starting to work with ESP micoprocessors - so please review - the change works fine with my esp32-s3, though.

Thanks for this!


Add support for specifying a migration template

I'd like to use this with TypeScript, and so having a way to provide my own template file (e.g. in a migration-template.ts) through a CLI parameter would be awesome.
For example the template I'd use would be:
import { MigrationBuilder } from 'node-pg-migrate';

export function up(pgm: MigrationBuilder) {
}

export function down(pgm: MigrationBuilder) {
}

Any concerns / thoughts?

I have little experience with TypeScript, but doesn't it also need some typescript definitions for node-pg-migrate? (https://github.com/DefinitelyTyped/DefinitelyTyped)
I need to check what are best practices / standards for these type of things...

Yep you're correct :smile: I'm planning to contribute some types to DefinitelyTyped for this, I'm fleshing them out at the moment as I write migrations.

I've raised https://github.com/DefinitelyTyped/DefinitelyTyped/pull/17875.

@dolezel @bradleyayers this issue should be closed as solved by #165

Great — thank-you!


Cherry-pick ES6 targeting and big-integer addition to master

Fixes #2207
Specific Changes

Updated packages to target ES6 (#2204)
Switch adaptive-expressions to es6 (#2205)

Pull Request Test Coverage Report for Build 129245

8 of 10 (80.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 80.074%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/adaptive-expressions/src/expressionFunctions.ts
8
10
80.0%

Totals

Change from base Build 129003:
-0.004%

Covered Lines:
12661

Relevant Lines:
15090

💛  - Coveralls


Add multipage business cards test form recognizer

Add tests to cover scenarios for multipage documents with one business card per page for business card recognition API in Form Recognizer.
Update test methods to combine receipt and business card testing methods.

/check-enforcer override


Exception when using the example-helper

I was trying to use the example helper to include example-code into the README.md
of this project
I get an exception and I do not see the reason.  Can you tell me what I am doing wrong?
The exception is:
TypeError: undefined is not a function
    at DestroyableTransform._transform (/home/nknappmeier/programs/node/node-v0.12.0-linux-x64/lib/node_modules/verb-cli/node_modules/verb/lib/plugins/dest.js:44:26)
    at DestroyableTransform.Transform._read (/home/nknappmeier/programs/node/node-v0.12.0-linux-x64/lib/node_modules/verb-cli/node_modules/verb/node_modules/through2/node_modules/readable-stream/lib/_stream_transform.js:184:10)
    at DestroyableTransform.Transform._write (/home/nknappmeier/programs/node/node-v0.12.0-linux-x64/lib/node_modules/verb-cli/node_modules/verb/node_modules/through2/node_modules/readable-stream/lib/_stream_transform.js:172:12)
    at doWrite (/home/nknappmeier/programs/node/node-v0.12.0-linux-x64/lib/node_modules/verb-cli/node_modules/verb/node_modules/through2/node_modules/readable-stream/lib/_stream_writable.js:237:10)
    at writeOrBuffer (/home/nknappmeier/programs/node/node-v0.12.0-linux-x64/lib/node_modules/verb-cli/node_modules/verb/node_modules/through2/node_modules/readable-stream/lib/_stream_writable.js:227:5)
    at DestroyableTransform.Writable.write (/home/nknappmeier/programs/node/node-v0.12.0-linux-x64/lib/node_modules/verb-cli/node_modules/verb/node_modules/through2/node_modules/readable-stream/lib/_stream_writable.js:194:11)
    at Stream.method [as write] (/home/nknappmeier/programs/node/node-v0.12.0-linux-x64/lib/node_modules/verb-cli/node_modules/verb/node_modules/event-stream/node_modules/duplexer/index.js:47:39)
    at Stream.ondata (stream.js:51:26)
    at Stream.<anonymous> (/home/nknappmeier/programs/node/node-v0.12.0-linux-x64/lib/node_modules/verb-cli/node_modules/verb/node_modules/session-cache/node_modules/continuation-local-storage/context.js:74:17)
    at Stream.emit (events.js:107:17)

I think it's failing because of the .txt extension, sorry about that. when I created it my use case was to use working javascript examples. let me know if it doesn't work with just the javascript one.

It also did not work if I removed the "example.txt" line completely. I have now written a small helper myself, which does work for me. I am still wondering. Is the example-helper doing the same thing or have I misunderstood the use-case:
verb.asyncHelper("inline_file", function (filepath, next) {
    var filename = require.resolve(filepath)
    var moduleName = this.context.name;
    return fs.readFile(filename, 'utf8', function(err, contents) {
        if (err) {
            return next(err);
        }
        // This is the module-name that is used by the example file, when
        // requiring the current module
        var requirePattern = path.relative(path.dirname(filename),__dirname);
        var requireRegex = new RegExp("require\\([\'\"]"+requirePattern+"/?[\'\"]\\)","g");
        next(null,contents.replace(requireRegex,"require('"+moduleName+"')"));
    });
});

I am still wondering. Is the example-helper doing the same thing or have I misunderstood the use-case:

I've been using it to just include a javascript snippet from example.js. Since example.js is supposed to demonstrate working code (for my use case), the only additional thing the helper does is replace
var foo = require('./');

with
var foo = require('foo');

So that it makes sense from the frame of reference of a readme example. which is what you're doing as well.
My only feedback is that it might be easier/less code to register as a sync helper. The only reason I register it as async is to use async-helper-base 1) since it's already there, and 2) since I'm pre-loading the javascript files so they can be pulled off the views object as templates. Really, that's not necessary, the way you're doing it looks great too

Well, now that you mention var foo = require('./');: The example helper does work, if the example file is in the project's root-directory.

you might be able to change the cwd by passing it to the helper:
{%= example("foo", {cwd: "bar") %}

OK, but when I have an example in a subdirectory of the project, I have to do require('../') instead of require('./'), so the example-helper won't substitute the module name.
However, I was wondering, how the whole helper setup in verb works. I was trying to figure out what to do from the code, but the source code of the example helper looks like it would just return the string (which in my understanding would be the filename) in a slightly modified form. I came across some more complex files when using a debugger, but I couldn't really grasp the whole concept.
Could you write a little about that?

so the example-helper won't substitute the module name.

{%= replace(example("foo", {cwd: "../"), "../", name) %}

Is there a list of helpers that can be used with verb without including any other packages?

let me know if this helps! https://github.com/verbose/verb/commit/892cee09c0bc4476985aaf1695f4e3a6d993d7b9

Looks great, thanks

no prob, happy to help

Just to let you know. I change the verb-template of line-counter to use
{%= embed("./examples/example.txt") %}

and
{%= replace(embed("./examples/regex-matches.js"),"../",name) %}

Now there is no need for a verbfile.js and custom helpers anymore.
I like verb... It's exactly what I was looking for. Sadly, it took a while to find it...

Awesome!
Sent from my iPhone

On May 13, 2015, at 2:38 AM, Nils Knappmeier<EMAIL_ADDRESS>wrote:
Just to let you know. I change the verb-template of line-counter to use
{%= embed("./examples/example.txt") %}
and
{%= replace(embed("./examples/regex-matches.js"),"../",name) %}
Now there is no need for a verbfile.js and custom helpers anymore.
I like verb... It's exactly what I was looking for. Sadly, it took a while to find it...
—
Reply to this email directly or view it on GitHub.


Centralizing the site

What do you think about centralizing the site ?
today

centralizing

That's the direction I want to take for the page design. It's not necessary to center the navigation bar if you want to open a PR

fixed by #1670


Client gen

I didn't know spaces were valid in makefiles. I thought it had to be all tabs.
Separated in two commits. one of gencode.

Followed this document https://github.com/kubernetes/community/blob/master/contributors/devel/generating-clientset.md

We can use the flag --clientset-name to change the internalclientset directory. The possible naming scheme there seems to indicate release versions.

seem pretty straightforward
LGTM

seem pretty straightforward

I think you mean 'pretty standard'

@arschles has given me verbal LGTM on this

+1
Sent from my iPhone

On Feb 3, 2017, at 15:47, Paul Morie<EMAIL_ADDRESS>wrote:
@arschles has given me verbal LGTM on this
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


BUG:Inconsistent Button Arrangement for Login and other option  in Navbar

Is there an existing issue for this?

[X] I have searched the existing issues

What went down?

The Navbar currently features multiple buttons, including "Home," "About Us," "Services," "Careers," and more. However, there is an inconsistency in the arrangement of the "Login" and "Sign Up" buttons. These buttons should be grouped together or placed prominently on the Navbar for better user accessibility and a more intuitive interface.
Record

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

Please assign it to me, i can fix this

@10kumaranurag01 Issue assigned to you


fix for confluent-plaform 5.4 issue

Fixed the issue on confluent-plaform 5.4.0

great you figured it out, I think I saw similar way for others to use it
perhaps another sanity check would be to see any incompatibilities between versions of netty in the driver and in confluent, to be 99% nothing breaks


Only generate a new image on demand if pre_cache is disabled

Hi,
I have a remote storage backend that uses SFTP for file operations (very slow) and HTTP for getting the file itself (very fast). Due to the slowness of the backend, I have set pre_cache=True so that after an image has been processed (thumbnails created, etc.), I can avoid calling SFTP operations as much as possible for performance reasons. Even calling exists() has a noticeable impact on performance.
I've noticed that ImageSpecFile.generate() is called every time the model's url() or _get_file() is called. Because generate() does an exist() call on the storage backend, I have disabled generating the image if pre_cache is True.
I'm not sure if my fix is the right one for this problem, but it seems to work for me.

Yeah, the performance hit is obviously unacceptable. I think your approach is probably the right one, though right now there is a guarantee that the image will always exist when accessed.  Removing that guarantee may throw people off at first, but I don't see any way around it.

We're going to have to solve this issue, #74, and #75 simultaneously. If we delete the cached image on every save (and I'm struggling to figure out a way to avoid it), and the image isn't necessarily automatically regenerated (as a97e764 proposes), there will be times when the spec image just doesn't exist.

#89 is the general-purpose solution we've come up with, so I'm going to close this.
@mkai It would be super awesome if you could contribute an image cache backend that would fit this kind of situation.

@matthewwithanm Perfect solution. Thanks for your great effort! Just migrated to v2.0a.
The supplied NonValidatingImageCacheBackend fits my use case (the one mentioned in the issue) entirely because it creates the thumbnails exactly once upon saving the model field. I have no problem to assume that nobody ever changes the original images on the file system outside of Django, so there's no need to do future checks if a thumbnail needs to be regenerated in the future. If I'm missing something here, please tell me :)

You've got it exactly right! Good to hear it's going to work for you, and thanks for your help!


Environment setup

Set up dependencies for the project
JUCE - https://docs.juce.com/master/tutorial_new_projucer_project.html
Stockfish - https://github.com/official-stockfish/Stockfish

I propose we all try to get this example application (which includes a GUI and audio) compiled and running on our machines by Wednesday.


Consolidate Tracker Core module into Java Tracker

As said in issue #115 I have refactored the project structure :

The project is no more splitted in submodule (I removed the core module).
One Tracker test was not portable (timezone).
Tests have been switched from junit 3 to junit 4.

For this commit I have keep the namespace com.snowplowanalytics.snowplow.tracker.core.
If your Ok I will remove this namespace in another pull request.
I also wonder why there is a war packaging (I'm not familiar with android world)
ping @epantera @lixiaoyi

Thanks @dstendardi - this all looks good. Please go ahead and change the namespace to com.snowplowanalytics.snowplow.tracker, dropping the core.
And yep you can remove the war packaging (if it's needed for Android, it will live in snowplow-android-tracker)...

Updated title

Scheduling - this will be 0.7.0...

core package removed
war packaging removed

Huge thanks @dstendardi - I am closing this now as this is fully merged into https://github.com/snowplow/snowplow-java-tracker/commits/release/0.7.0, which is the next release and will go out very shortly...


[feat] Synchronous variants to path functions

Describe the problem
All dir functions in path (homeDir, configDir, etc.) are asynchronous. This is great as a default behaviour, but sometimes it's extremely inconvenient.
I'm currently building an app where I define the application's path based on the user's configuration directory and then use it in many different functions, including some synchronous ones. With the current library, this is completely impossible as getting the user's config. directory results in a Promise. So I'm forced to make all these functions asynchronous just because of that.
// This is a Promise<string>, I'd like a string.
export const DATA_DIR = path
  .configDir()
  .then((configDir) => path.join(configDir, 'Writer'))

Describe the solution you'd like
Would it be possible to make synchronous counterparts of these methods? This would be quite useful in this scenario, and fetching these directory's path is extremely fast so it wouldn't make the app unresponsive.
Alternatives considered
No response
Additional context
No response

this is not possible currently due to some limitation, subscribe to https://github.com/tauri-apps/wry/issues/454 for when it is resolved.


fix(server): remove command limitaion

Reason for This PR
issue #511
Description of Changes

Remove command arguments check.
Leave the check of command args number. The minimum command should contain 2 args: <executable> <script> separated by space.

License Acceptance
By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.
PR Checklist
[Author TODO: Meet these criteria.]
[Reviewer TODO: Verify that these criteria are met. Request changes if not]

[x] All commits in this PR are signed (git commit -s).
[x] The reason for this PR is clearly provided (issue no. or explanation).
[x] The description of changes is clear and encompassing.
[x] Any required documentation changes (code and docs) are included in this PR.
~- [ ] Any user-facing changes are mentioned in CHANGELOG.md.~
[x] All added/changed functionality is tested.

linked #368


User datatype requires Generic instance

The README example should be updated accordingly

If this is about the example
{-# LANGUAGE OverloadedStrings #-}

import Control.Applicative
import qualified Data.ByteString.Lazy as BL
import Data.Csv
import qualified Data.Vector as V

data Person = Person
    { name   :: !String
    , salary :: !Int
    }

instance FromNamedRecord Person where
    parseNamedRecord r = Person <$> r .: "name" <*> r .: "salary"

main :: IO ()
main = do
    csvData <- BL.readFile "salaries.csv"
    case decodeByName csvData of
        Left err -> putStrLn err
        Right (_, v) -> V.forM_ v $ \ p ->
            putStrLn $ name p ++ " earns " ++ show (salary p) ++ " dollars"

I don't see the problem; the example compiles just fine.


mirgation d3 to latest version

Do you plan to update D3 to 5 version?

While we are still using react-d3-components here, we don't plan to upgrade except if somebody sends a PR for it.


Not available on hex?

Hi, I'm interested in using this library but it seems like it hasn't been published to hex yet. Why not? Or is it and I just can't find it?

We have not published this package to https://hex.pm yet.
Will update the Readme once it is published to Hex.
You can use this package via the github option in mix.exs file.
{:firebase_admin_ex, github: "scripbox/firebase-admin-ex"},

Thanks for the info! What is the rational for not having published yet? Is it just documentation or are the apis going to change significantly? Should I be cautious about using this library in production?

Thanks for the info! What is the rational for not having published yet? Is it just documentation or are the apis going to change significantly? Should I be cautious about using this library in production?

Done - https://hex.pm/packages/firebase_admin_ex


kubectl's installation should be cached

...so that it doesn't need to be installed on every build.

It seems all that is done on CircleCI is to update kubectl on each run. Looking through some CI runs it takes about 10-20 seconds.
https://github.com/reactiveops/k8s-scripts/blob/adfdd888270face050f8a7b92d11d6a5c15ffc7e/bin/ensure-kubectl#L53
How long are you seeing runs taking?

30s. https://circleci.com/gh/reactiveops/krails/180
Not huge, but for a 3:09 build ~1/6 is significant for a developer.
I'm thinking that just the kubectl bin can be put in to something like ~/. kube (via ensure-kubectl) and then circle can be told to cache that directory across builds. ensure-kubectl can then check for kubectl's existence and perhaps the right version.
That should get it down to 1s for a cache hit

Gotcha. Although there are no tests at the moment so it would likely be longer and not quite 1/6. At the moment the builds I'm working on are ~25, so not quite the biggest issue for them at the moment.

Now that we've moved to using ci-images with everything effectively cached, this seems to have been resolved.


Fixed exports in type declaration file

I ran into issue when I tried to use the new type declarations. Example code:
import { router } from 'fs-router'
let intents = router(__dirname + '/intents')

Typescript was able to compile this, but running the code caused this error:

TypeError: fs_router_1.router is not a function

I realized this was because the library uses this syntax:
module.exports = function router() {

Instead of this syntax:
module.exports.router = function router() {

I made these changes to support that fact.
Incidentally, with this change, my code no longer compiled. Typescript returned an error like

Module '${my app path}/node_modules/fs-router/index"' resolves to a non-module entity and cannot be imported using this construct.

I modified my usage to this:
import Router = require('fs-router')
let intents = Router(__dirname + '/intents')

And now everything seems happy.

Coverage remained the same at 100.0% when pulling 7e48b3fad1ec6b66f0a40c5e2c6283891e37bbe9 on michaelmosher:master into 5e1638ddb2f58027ce4243ac38602a361b1f631e on jesseditson:master.

Coverage remained the same at 100.0% when pulling 7e48b3fad1ec6b66f0a40c5e2c6283891e37bbe9 on michaelmosher:master into 5e1638ddb2f58027ce4243ac38602a361b1f631e on jesseditson:master.


Performance tweak about src-map emission

Just doing some repo cleanup for better focussing on active work:
Closing PR as it seems inactive for a longer time. Please re-open if it is still relevant and after it has been updated with latest master changes.


[Windows] Remove LLVM standalone installation

LLVM is already included in the Visual Studio installation. This duplication creates conflicts between versions. The current MSVC C++ standard library requires at least Clang 17, while the standalone LLVM is 16.
There is no reason to include two different LLVM versions. Use the one from the Visual Studio installation. Should anyone need standalone LLVM, they can install it with a single command: choco install llvm.
Fixes: #10001
Check list

[x] Related issue / work item is attached
[x] Tests are written (if applicable)
[x] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

Closing, as it seems to be not wanted change. I still think duplicating software with conflicting versions is not making things easier.


Feature/disable strict mode in the sizes api endpoint

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Use a variable for the Grafana Docker image tag

:tada: This PR is included in version 2.0.0-next.22 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Analysis refactor master

Previous PR was done against the wrong branch.
This PR syncs the analysis-refactor branch with the current status of master. I will comment in the code about the specific things that I would like people to take a look, since most of the things here are actually changes that have been already merged in master.

This is not in sync with the latest master!


Create Obscure Functionality

Diary Bot Function: Obscure
Create the functionality to meet the functional specifications for obscure to obscure relationships or entries initially by substituting proper names with their first letter.
Member Version

[x] routes.mjs create members obscure
[x] functions create obscure
[x] mylife-avatar.mjs: obscure()
[x] mylife-agent-factory.mjs

[x] get item
[x] send summary to LLM, ask for coding in response? [i.e., who is who?]; ergo factory has correct mbr_id and has access to generic LLM in modular code
[x] validate response
[x] process response
[x] save response
[x] return instructions and saved item

API  Version

[x] routes.mjs create api obscure
[x] api-functions.mjs create obscure: validate, extract, and pass to MyLife
[x] core.mjs class MyLife

[x] create proxy Member Avatar (bot-factory for now?) to address obscuration; note: interestingly it looks like the proxy Bots (which occur understandably in factory, i.e., not to be accessed without trace) all require and are infused with the modular Generic Version as I can't tell if it's overwritten, but might be able to remove property this.#llmServices in all factory instances (and just rely on modular version when it is appropriate)
[x] call obscure on bot-factory

[x] Diary Bot yaml for API

Bot Version

[x] diary bot instructions should include the option to obscure which requires the itemId (see comment)
[x] define obscure function in globals.mjs (see comments)
[x] mylife-llm-providers.mjs add function as a callback
[x] note: of course bots could engineer this on their own with the CRUD ability - question is just that: afford vanilla with options to override
[x] side quest: general System Bot

update diary-intelligence-1.0.json
### OBSCURE ENTRY\nWhen request is prefaced with `update-request` it will be followed by an `itemId`.\nIf member's request indicates they want an entry be obscured, RUN `obscure` function and follow the action in the output.\n

obscure: {
	description: "Obscures a summary so that no human names are present.",
	name: "obscure",
	parameters: {
		type: "object",
		properties: {
			itemId: {
				description: "Id of summary to obscure",
				format: "uuid",
				type: "string"
			}
		},
		required: [
			"itemId"
		]
	}
},


Feature Request: Automatic Refresh

Currently I have to hit the refresh button to check on the state of my automated tests. This is a huge barrier to keeping my full PR flow within VSCode, especially since I have to scroll to the top to hit refresh, then scroll all the way back down. In the browser these updates are automatic and don't need a full page refresh.
As a stopgap, a hotkey (ctl/cmd+r?) would make the experience much smoother than it is now.

+1

I believe you can create a hotkey for that because a command is exposed under Refresh Pull Requests.

Noted in scenario https://github.com/github/codespaces/issues/1608

This is now working in 2020.11.3841 (Nightly)


Integration of KFNBC UI into backend KFNBC

[x] KFNBC (Samu) is work in progress and can be used
[x] Install KFNBC KfDef if Maro’s cluster (Samu will work with Maros on this).
[x] Create new feature flags
[x] Include backend APIs
[x] Create route to new spawner and hide it

Fixed by https://github.com/opendatahub-io/odh-dashboard/pull/245


Can I have Better Instructions on how to Install this on a Debian GNU/Linux 9 os please and thank you.

Can I have Better Instructions on how to Install this on a Debian GNU/Linux 9 os please and thank you.

Hello
i m stuck
First i do that :
http://<IP_ADDRESS>/MCHostPanel/install.php
but when i try to log,
i can log, and no message is show ;
i can pass this :

what is wrong please :
how debug please

i found solution here 👍
https://github.com/Alanaktion/MCHostPanel/issues/41
works for me
thanks


Break connection between graphql nodes

Is your feature request related to a problem? Please describe.
I would like to break the connection between two graphql nodes without deleting one of them.
Describe the solution you'd like
I would like to break the connection setting one of the two connection id to null

@AlessandroAnnini do you want to do this on a per-record basis or remove all connections between two models?
Just FYI that if you're using Dynamo as your backing data store, the latter (if not rate limited) will require an expensive scan of your entire data store (on both tables), followed by an update of every item in each data store. Not sure if it's inside the scope of the Amplify CLI to pre-warm your capacity for you and spin it back down after the connection-deletion operation is completed, but I believe that would be required to implement your feature request.


Videoclips issue

When running the video_demo.py, I encounter the error  'av.container.input.InputContainer' object has no attribute 'close'. I assume the reason is VideoClips  in the file data_loader.py. (My torch version is 1.2.0)

see https://github.com/pytorch/vision/issues/1246


PoB 1.4.65 import error

I'm unsure if this is related to the maintenance currently going on, but since the websites working I assume that isn't the case.
Upon trying to import a character its throwing an error. I can choose to ignore the error and it claims success, but nothing actually imports. Restarting app does not fix it.
Including screencap

I am having the same issue.

Importing character with account name: d347hm4n

Same issue as above users. It happened when importing Items and skills

Fixed in the next update.

Thank you for the fix

Fixed for me thanks!


[SPARK-46368][CORE] Support readyz in REST Submission API

What changes were proposed in this pull request?
This PR aims to support readyz in REST Submission API as a meaning of state != RecoveryState.STANDBY.
readyz semantic is defined by K8s documentation.

https://kubernetes.io/docs/reference/using-api/health-checks/

Machines that check the healthz/livez/readyz of the API server should rely on the HTTP status code.
A status code 200 indicates the API server is healthy/live/ready, depending on the called endpoint.

Why are the changes needed?
Previously, the users can check the Spark Master pods' readiness like the following.
readinessProbe:
  exec:
    command: ["sh", "-c", "! (curl -s http://localhost:6066/v1/submissions/status/none | grep -q STANDBY)"]

After this PR, we have the following result.
200 OK
$ curl -vv http://localhost:6066/v1/submissions/readyz
< HTTP/1.1 200 OK
{
  "action" : "ReadyzResponse",
  "message" : "",
  "serverSparkVersion" : "4.0.0-SNAPSHOT",
  "success" : true
}

503 Service Unavailable
$ curl -vv http://localhost:6066/v1/submissions/readyz
...
< HTTP/1.1 503 Service Unavailable
...
{
  "action" : "ErrorResponse",
  "message" : "Master is not ready.",
  "serverSparkVersion" : "4.0.0-SNAPSHOT"
}

Does this PR introduce any user-facing change?
No, this is a new API.
How was this patch tested?
Pass the CIs.
Was this patch authored or co-authored using generative AI tooling?
No.

Thank you, @viirya .

Let me merge this because the last commit is about a comment change.


feat: Window resizing support

Currently due to the defaulting decoration-less behavior of verso, we lack several window features including the ability for users to resize the window dynamically on Windows and Linux (macOS works fine). Adding window resizing support will improve user interaction, allowing users to adjust the browser window size to their preference by dragging the window edges or corners.
Expected Behavior:

Users should be able to resize the window by clicking and dragging the edges or corners.
The window size should adjust smoothly to match the mouse movements.
Resizing should function properly on all supported operating systems (Windows and Linux).
Supports both horizontal and vertical resizing, as well as corner-based diagonal resizing.

Technical Approach:

Enable resizing functionality through native window APIs.
Monitor mouse events (mousedown, mousemove, and mouseup) around the edges and corners to trigger resizing.

Acceptance Criteria:

[ ] Users can resize the window by dragging its edges or corners.
[ ] Resizing behaves consistently across different operating systems.
[ ] Display corresponding cursor icon when on edges or corners.

Good To Haves:

To ensure resizing actions are smooth and responsive to user input.
To ensure that minimum and maximum window size constraints are respected during resizing.
To ensure compatibility with other UI elements, such as buttons and menus, to prevent unintended resizing interactions.
Resizing may cause flickering to the window itself which should be addressed in the future

Add double click on panel to toggle maximize window (still in progress, will keep update here)
https://github.com/versotile-org/verso-panel/pull/8

Although macOS works fine by default in resize.
We should make sure the behaviour is still the same after our implementation.
I tried to test it after the commit of resizing PR and it seems to still work fine on msOS:
https://github.com/user-attachments/assets/7f0cb785-bbc5-49e9-b597-dbafb4fa1630

Wayland:
https://github.com/user-attachments/assets/386013d4-6520-460e-9918-dbfe37490737
X11:
https://github.com/user-attachments/assets/8b9bea7a-2da5-440f-a827-04aec2b94e5f

Wayland:
https://github.com/user-attachments/assets/02cd40f1-989a-4834-a412-dfa3d19e0908

Here's windows' resizing. Note that the black background is due to opengl from my device. Wgl can display transparent background.
Perhaps we could make the gl more configurable in the future.
https://github.com/user-attachments/assets/eded7b2b-d99c-4c3a-aa78-438f77fc4048

Linux:
https://github.com/user-attachments/assets/092c72e0-36f9-4ca1-bf67-7eca990af136

I'm working on smooth resizing for weeks. And here are some solid result on Linux. I'll send the PR once I make sure all platforms are smooth.
Screencast from 2024年11月05日 20時45分20秒.webm

And here's macOS:
https://github.com/user-attachments/assets/16f60c7a-17e1-4529-a3ac-09603a751860

Forgot about Windows which doesn't have the stutter in the first place already.
https://github.com/user-attachments/assets/c0816ae6-128b-498c-a6ba-24ed6831acd0


Fix warnings and make project consistent

Analyzing the repo I've noticed a few things which might be worth to fix.

Fix over 60 warnings.
Consolidate the Microsoft.NETCore.App package for the projects. Currently the projects using versions 2.0.0, 2.1.0, 2.2.0.
Rename the folder  GitReleaseManager to GitReleaseManager.Core to match the project name.
Move Extension classes to the Extensions folder
Some unit test have the method format ThisIsATest() and others This_Is_A_Test().
Suppres the SA1310 warning in GlobalSuppressions.cs (in this case for const string MY_VALUE).

Make the project consistent with other GitTools projects (e.g. GitVersion)
// <copyright file="file.cs" company="GitTools Contributors">
//     Copyright (c) 2015 - Present - GitTools Contributors
// </copyright>
//-----------------------------------------------------------------------

Remove the file header
Classes in the GitVersion project don't contain any file header. A class rename don't change the filename in the header, which cause a warning and increases the maintenance effort.
using definitions above the namespace
Visual Studio creates the using definitions above the namespace, so currently new definitions must always manually be placed below the namespace.
Consistent folder naming
The folders in the project should be lowercase

Icons -> icons
Source -> src

Move the file /Documentation/Legal/CREDITS.md to the root folder.
The /Documentation folder is also misleading, because it contains no docs. The project docs are in the docs folder.

I have also a question of understanding. The projects GitReleaseManager.Cli and GitReleaseManager.Tool both create a GitReleaseManager.exe. Through the link of GitReleaseManager.Cli classes the GitReleaseManager.Tool causes warning because of the inconsistent namespace. What ist the difference between this two projects?

@akordowski GitReleaseManager.Cli creates the full .Net Framework version of the application, with the GitReleaseManager.exe, and which is ultimately published to:
https://www.nuget.org/packages/gitreleasemanager
https://chocolatey.org/packages/gitreleasemanager.portable
The GitReleaseManager.Tool creates a .net global tool version of the application, which is ultimately published to here:
https://www.nuget.org/packages/gitreleasemanager.tool

@akordowski I will have to go through the list of items above in more detail.  Some of them I have no issue with being undertaken, others will need some more thought.

Thanks for the clarification!

Another issue I forgot.
Unify the config extension
The config for the GitVersion project has the file extension .yml and the GitReleaseManager the extension .yaml. I would suggest to unify this aswell and use the .yml extension. The GRM should then checkt for both extensions.

Associated issue: https://github.com/GitTools/GitReleaseManager/issues/269

@akordowski I have been thinking some more about some of the issues that you have raised in this issue.  I need to think about them some more, and I will update this issue with some thoughts once I have put them all together.

@gep13 Alright. Take your time.


Ikea trådfri?

Is the adapter working with Ikea trådefri?
Hardware: RPI 3 and dresden usb stick (ConBee)
I have setup deconz software and linked three device (light bulb, motion sensor and switch) and i see them in deconz we-gui.
I have the deconz adapter loaded and configured.
There is only one objeckt in instances tab.

In the script tab I can see all three objects,

There are five objects because i renamed  two of them in deconz!
I need help!
Its ok to answer in German.(google translate works ok)

Sorry for the late answer.
I think in actual Version of the Adapter all Kind of devices is supported if it is in deConz.
You habe to reload the object tree if new objects where added.
At this time objects have to have to be deleted by Hand.


Change text style (EXPOSUREAPP-4489)

change text style from body1 to subtitle as requested in ticket
some refactoring of the layout to better be able to identify the screen and text elements in preview

Link back to original issue https://github.com/corona-warn-app/cwa-app-android/issues/2013.


Copy-sample-jobs deleting

Copy-samples-job is deleting after completion (assuming it is due to ttl setting).  CICD leverages that job spec (copy and reissue) to force a copy of fresh notebooks for regression testing.  If the job deletes, the regression pipeline fails.  We need to investigate a better process (isolating a separate pipeline to redeploy charts?, making the ttl a customizable param in plugins?, etc)....

Due to new update to v1.20 for kubernetes, we suspect that the TTL now works


Split mode should be available in non full screen

Split mode is available only in fullscreen. It should be available within the container too.

Duplicate of https://github.com/Ionaru/easy-markdown-editor/issues/21 ?

Try sideBySideFullscreen option. If set to false, allows side-by-side editing without going into fullscreen. Defaults to true.


fix: ensure clusters order in envclusters openapi

What's the purpose of this PR
XXXXX
Which issue(s) this PR fixes:
Fixes #5276
Brief changelog
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Read the Contributing Guide before making this pull request.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit tests to verify the code.
[ ] Run mvn clean test to make sure this pull request doesn't break anything.
[ ] Update the CHANGES log.

Summary by CodeRabbit

Bug Fixes

Improved the handling of property sets to preserve the order of insertion when converting lists to sets.

I have read the CLA Document and I hereby sign the CLA


Change healtcheck port into address format, redo #1197

As discussed in the comments in PR #1197. Also add documenation accordingly.
One thing I'm not sure about is the simple check in health.go if the address is usable in the GET request or not. From reading https://pkg.go.dev/net#Dial it seems that the only non-standard address format that would work in the net package but not in a GET url would likely only be :port, as the others listed here are actually also valid urls:
For TCP, UDP and IP networks, if the host is empty or a literal unspecified IP address, as in ":80", "<IP_ADDRESS>:80" or "[::]:80" for TCP and UDP, "", "<IP_ADDRESS>" or "::" for IP, the local system is assumed.
One additional thing I noticed is that while WOODPECKER_SERVER_ADDR and WOODPECKER_SERVER_ADDR use the default value format of :PORT, WOODPECKER_SERVER actually uses localhost:9000. I guess it makes a bit of sense, considering the server might not be local to the agent, but it looks a bit inconsistent this way. I don't think it would hurt to make the WOODPECKER_HEALTHCHECK_ADDR in this format too, but then it's different from the server flags again... :-)

Codecov Report
Base: 42.35% // Head: 42.33% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (7073fc9) compared to base (482678d).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1423      +/-   ##
==========================================
- Coverage   42.35%   42.33%   -0.03%
==========================================
  Files         153      153
  Lines       10488    10493       +5
==========================================
  Hits         4442     4442
- Misses       5766     5771       +5
  Partials      280      280

Impacted Files
Coverage Δ

cmd/agent/agent.go
0.00% <0.00%> (ø)

cmd/agent/health.go
9.52% <0.00%> (-1.29%)
:arrow_down:

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

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


Added the HorizontalIconAlignment property to the button.

Added the HorizontalIconAlignment property to the button.
You can now choose, if you want to show the icon left or right to the content.

Pull request type

Please check the type of change your PR introduces:

[ ] Update
[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[x] Documentation content changes

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Hm, the weird "alignment" usually should like it is placed inside of a space. Like: text on a page is aligned inside/between the borders of the page. Not next to the page.
Here, is more like a position relative to something else.

 Like: text on a page is aligned inside/between the borders of the page. Not next to the page. Here, is more like a position relative to something else.

But it isn't relative, that's the difference.

What I mean is, the icon is placed to the left/right of the content/text. Therefore I think "HorizontalIconPlacement" would be a good (or even more accurate) wording.
By the way, I assume this works well in RTL language?

What I mean is, the icon is placed to the left/right of the content/text. Therefore I think "HorizontalIconPlacement" would be a good (or even more accurate) wording.
By the way, I assume this works well in RTL language?

In this case I followed the standard alignment that MS uses.

#1120 & #887 can check the last commit.

I don't know why, but my vs is bugged rn. I cannot create visible styles and my formatter does instead of formatting only opened files all .xaml files.
Sorry for that.


Python3 support

It would be nice to Mongothon had python3 support.
Thanks!

Anyone tried using Mongothon for python3 yet? I need a good ORM and this one looks like a winner besides the fact the current support is python2.7. Thanks!

Hello,
I tried and with some adjustments in module loading and metaclasses, it
seams to work.
I did some changes on my computer. The problem is I lack of time to make
them clean and push them. :/
I'll inform you if I've time to do it.
Regards,
Azollyx.
2016-10-19 17:00 GMT+02:00 Jon Major<EMAIL_ADDRESS>
Anyone tried using Mongothon for python3 yet? I need a good ORM and this
one looks like a winner besides the fact the current support is python2.7.
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gamechanger/mongothon/issues/56#issuecomment-254839593,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOzqWXMUDDYC8yzCh3Eg8DgGuu1leZmYks5q1jCGgaJpZM4JkkSj
.

I'm wondering - is this something that is being planned for this project? I'd really like to use it in my project, but reluctant to do so because I generally prefer to use Python 3 in my projects.
Thanks!
Omri

Any update?

No sorry and I abandoned the project (not because the project is too
difficult but because I've too many things to do). Good luck.
Le dim. 31 mai 2020 à 17:31, Felix Isaac<EMAIL_ADDRESS>a
écrit :

Any update?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gamechanger/mongothon/issues/56#issuecomment-636486952,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADWOUWLSVLZ7CFFBMNV3YILRUJZ6PANCNFSM4CMSISRQ
.


A Responsive Portfolio Website using HTML, CSS and bit of JavaScript

please assign me this.Would love to work on this

It's already being made if you can enhance it then go ahead @bugsyy157
Portfolio : (Code)[https://github.com/keshavgbpecdelhi/Web-Development/tree/master/css-7]

Sure :)


Transport position is resetting to zero when loop region is changed

Describe the bug
When Transport looping is enabled and a loop region is redefined (i.e., defined once, transport plays and stops, then loop region is redefined), the transport subsequently starts from zero even if it is set to a non-zero value.
To Reproduce
Here is a fiddle demonstrating the issue: https://jsfiddle.net/3srjLto2/3/
Steps to reproduce:

Set Transport looping to true and define a loop region.
Start and stop Transport
Define a new looping region (different start and end points)
Set Transport position to the start of the new loop region
Start Transport again

Transport position plays from 0 rather than the specified time.
Expected behavior
Transport should play from its current position without being reset to zero.
What I've tried
I can only get this to happen after playing and stopping the Transport at least once.
Oddly enough, in subsequent attempts (setting position and starting from that position), the Transport will sometimes play from the set position, and sometimes play from 0 (about 60/40, 60% of the time from the correct position).  However it always seems to fail in the case described by the fiddle above.
Another note is that you can set the transport position after it starts and it will still reset to 0, as seen here:
https://jsfiddle.net/0Ly7gxqa/5/

Hi. We also observed this and had to build in some guard conditions to stop events at 0:0 triggering when setting transport positions whilst playing. From recollection, I think if you set a position ahead of the current position then it does not rewind, but if you set a position before the current position, it appears to perform a rapid stop (which appears to be a async operation), followed by the repositioning...


Hootsuite profile dropdown text change

Might be worth adding a comment on the ticket to explain that social media posts created in the past will still use the old ID when you view them on the /review-disruption and /view-all-social media pages and that for testing new posts should be created.


Junoscan Account account view not working

https://junoscan.com/accounts/juno1srn2ujlz7lffa2rnsk7336uj83pc4jyqr538wp

I cannot scan my address for transactions. Thanks


Feature/docu

Deleted apache logo, renamed index-en to index, adjusted namespaces and removed local references from serializations.

ids:this was just a test for widocu behaviour. "ids:" is enough. Please change this and rerun the script, especially for the serializations.

The issues have beens solved and the docu looks good.


[Release] sdk/resourcemanager/containerservice/armcontainerservice/4.8.0-beta.1

https://github.com/Azure/sdk-release-request/issues/4942

@Alancere , can you resolve the conflicts?

Conflict resolved.

@Alancere , release request is held on.

Ready for release


Wrong API call parameters don't fulfill with an error code

This is not certain and needs to be tested. Conveying the reported bug:
Say we have an endpoint with parameters startDate and endDate. We're making a request to this endpoint with only the parameter coinId. The node doesn't respond with an error code.

Will check this out. Do we know what API? You can message me directly if you want

The parameters weren't required, so it shouldn't have returned an error code anyway. I was also told that it didn't respond either, but I doubt that without further information.


Update Resources Page

Adds two new blocks to the Resources page for the Public Key Tool as well as File Validator.
You can see in the screenshots below, the links the two different new block navigate to:

Fixes: #8591, #8593

I'm just noticing that "ReportStream File Validator" is in title case and "Manage your public key" is in sentence case. Just wanted to make sure this is what we wanted @audreykwr ?

Thanks, @etanb! Let's just leave it for now because I think that was mainly because we were treating the File Validator a proper noun, but it's tbd if it should stay that way. But it's fine for the time being.


Pin NLTK version to fix JAX ci

Description
NLTK's latest versions have introduced a bug resulting in JAX ci failures.
Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Infra/Build change
[ ] Code refractor

Changes
Pin NLTK version to 3.8.1 for CI tests.
Checklist:

[x] I have read and followed the contributing guidelines
[x] The functionality is complete
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

/te-ci jax

LGTM. Thanks for the fix!

An upstream fix to keep an eye on: https://github.com/Unstructured-IO/unstructured/pull/3512


build: bumped version to 0.3.2

Codecov Report

Merging #9 (ec36932) into main (5231e65) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #9   +/-   ##
=======================================
  Coverage   88.37%   88.37%
=======================================
  Files           1        1
  Lines         129      129
  Branches       38       38
=======================================
  Hits          114      114
  Misses         15       15

Flag
Coverage Δ

unittests
88.37% <ø> (ø)

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

Continue to review full report at Codecov.

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


Make opencv semi safe threaded

This is semi safe thread implementation, with 3 images I get something like that:
Search 17.327390566539222 iterations/sec, Render<PHONE_NUMBER>536914 iterations/sec
Search 17.327390566539222 iterations/sec, Render<PHONE_NUMBER>536914 iterations/sec
Search 17.32713107228414 iterations/sec, Render<PHONE_NUMBER>26144 iterations/sec
Search 17.32713107228414 iterations/sec, Render<PHONE_NUMBER>7424791 iterations/sec
Search 17.327044546405496 iterations/sec, Render<PHONE_NUMBER>119563 iterations/sec
Search 17.327044546405496 iterations/sec, Render<PHONE_NUMBER>119563 iterations/sec
Search 17.327044546405496 iterations/sec, Render<PHONE_NUMBER>119563 iterations/sec

Ah yes, I created pull request just to give you an idea how to speed rendering up


Instruct maker to send random probes every minute

Fix #1632.
As the maker sends random probes through the coordinator and collects the results, the maker's scorer will get updated over time.
We can then share the scorer file with other peers connected to the coordinator (10101 apps), which should improve their payment experience.
This last part is not included in this PR because I want to see how this works in production before we actually support the app pulling the maker's scorer file.

Based on this patch shared by Matt from LDK.

🤔 I guess it was the easiest to put that code to the maker, but IMHO it shouldn't be there.

Yeah, it should not be there, but probing is not exactly a heavy or risky activity, so I don't see any harm in putting this there temporarily.
As soon as we want to use the maker in a more interesting way, moving the code out is trivial.

Also would that work if the maker does not have a channel with the coordinator? or would we need to create a channel?

I need to test that. I would assume you do need a channel.


shared_preferences package: not suitable for tests

Please take a look at my test.
void main() async {
  TestWidgetsFlutterBinding.ensureInitialized();

  final store = AuthStore();
  GetIt.I.registerSingleton<ApiDio>(FakeApi());

  var sharedPrefs = SharedPrefs();
  await sharedPrefs.init();
  GetIt.I.registerSingleton<SharedPrefs>(sharedPrefs);
  SharedPreferences.setMockInitialValues({'token': '12345678'});

  test('token is not null after login', () async {
    when(GetIt.I<ApiDio>().dio.post(any, data: anyNamed('data')))
        .thenAnswer((_) async => Response(statusCode: 200, data: {
              'uuid': '16c29bd2-4f37-4c5e-906c-c0c41ea09922',
              'token': 'eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9',
              'role': 'user',
            }));

    var result = await<EMAIL_ADDRESS>'qwerty');

    expect(result, true);
    verify(GetIt.I<ApiDio>().dio.post(any, data: anyNamed('data'))).called(1);
    expect(store.isAuth, true);
  });
}

Where SharedPrefs is a simple wrapper for SharedPreferences. store.login() uses it internally.
import 'package:shared_preferences/shared_preferences.dart';

class SharedPrefs {
  SharedPreferences _sharedPreferences;

  Future<void> init() async {
    _sharedPreferences = await SharedPreferences.getInstance();
  }

  String loadToken() {
    return _sharedPreferences.getString('token');
  }
  ...
}

Expected results - test is passed.
Actual results: store.login() throws the following error:
Failed to load "D:\Flutter\bioniq\test\auth_test.dart": MissingPluginException(No implementation found for method getAll on channel plugins.flutter.io/shared_preferences)

✖ loading D:\Flutter\bioniq\test\auth_test.dart
Exited (1)

Can you please try adding flutter prefix as mentioned here
No, it's not related. I can even remove the whole "setMockInitialValues" string, and the error remains.

pubspec.yaml: https://pastebin.com/aKNwBGnZ
flutter doctor:

Hi @subzero911
I just tried to test a simple example and I realized you're calling setMockInitialValues after ensureInitialized, it should be called before.  Test runs with no issues on shared_preferences: ^0.5.10
test.dart
import 'package:flutter_test/flutter_test.dart';
import 'package:shared_preferences/shared_preferences.dart';

void main() async {
  SharedPreferences.setMockInitialValues({'token': '12345678'});

  TestWidgetsFlutterBinding.ensureInitialized();
  SharedPreferences sharedPreferences = await SharedPreferences.getInstance();

  test('test sharedPrefs', () async {
    var prefs = await SharedPreferences.getInstance();

    expect(prefs.getString('token'),
        '12345678'); //  change '12345678' to 'abc test will faill
  });
}

flutter doctor -v
[✓] Flutter (Channel stable, 1.20.3, on Mac OS X 10.15.6 19G2021, locale en-GB)
    • Flutter version 1.20.3 at /Users/tahatesser/Code/flutter_stable
    • Framework revision 216dee60c0 (8 days ago), 2020-09-01 12:24:47 -0700
    • Engine revision d1bc06f032
    • Dart version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/tahatesser/Code/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/tahatesser/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.7)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.7, Build version 11E801a
    • CocoaPods version 1.9.3

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 49.0.2
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.48.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.14.1

[!] Connected device
    ! No devices available

! Doctor found issues in 1 category.

I found that I must call SharedPreferences.setMockInitialValues({}), even with empty map. If I do not call it, i have a MissingPluginException.
I can do it after ensureInitialized, but it should be before SharedPreferences.getInstance(). if I do it before, i have the same MissingPluginException.

@subzero911
setMockInitialValues is called before instance in tests reference
Please try this test https://github.com/flutter/flutter/issues/65176#issuecomment-689359629

I confirm that this test is working.
But TestWidgetsFlutterBinding.ensureInitialized(); should be the first line, as stated in the docs, isn't it?

Hi @subzero911
Yes, it is but shared_preferences case, it seems to be that we need to add it before ensureInitialized. We could make it clear why in the doc issue https://github.com/flutter/flutter/issues/42780
Please add it before, your tests should work fine
given your last message I feel safe to close this issue, if you disagree please write in the comments and I will reopen it.
Thank yo


fix for transient failure in OffheapIncrementalIndex

before facts get it , indexAndOffsets should already know about it
fixes https://github.com/druid-io/druid/issues/2456

:+1: after travis

I will send backport PR to 0.9 once merged.

:+1: after adding a small comment in code about ordering.

@himanshug needs backport


docs: flash on nav items

    Flashing can prob be fixed later, its less annoying than the layout shift so this is good for now

Originally posted by @juliusmarminge in https://github.com/t3-oss/create-t3-app/issues/750#issuecomment-1317115060

Hi @juliusmarminge, can take on this task as a new contributor?

Go ahead 👍 I'll assign you

Hi @juliusmarminge, hope you're doing well. Is this issue fixed already? The flicker does seem to be as apprehensive in the current site as compared to the gif attached in the issue.

hi @umarfchy, I hope you have a nice day. The issue is still there, but you may have to open network in devtools and slow the loading of stylesheet to notice it. Btw 42069 stars nice 👍

I think it's fixed for the theme toggler but still there on the github icon

https://user-images.githubusercontent.com/51714798/203858754-c1008b6c-9da0-45ae-b6b0-859b4ec1a386.mp4

Got it. I'm on it. Thanks for the info 👍

Hi @juliusmarminge, Hope you're doing well. I think the issue has already been resolved by other changes. Unfortunately, didn't get much time thereafter to look into the issue. I'm unassigning myself from the issue. Thanks :)

Yep seems to be fixed!


[OSGi] Avoid importing javax.annotation to prevent problems from split packages

Commit be94e074b8856751f4f9e77c41d6045a1900abb2 marked a field as @Nullable, which caused the generated OSGi metadata to include an Import-Package: javax.annotation.  java.annotation.Nullable is one of the annotations proposed by JSR 305, and whose implementation is provided by the FindBugs project in a OSGi bundle called org.jsr-305.  Unfortunately other bundles define javax.annotation and so we have a split package situation, which is never fun.
This patch adjusts the OSGi metadata-generation process to not remove the javax.annotation package from the required imports.  It turns out the JSR 305 annotations don't actually have to be present at runtime.  The only consequence  is that any attempts to fetch the annotations at runtime via java.lang.reflect will not see the JSR 305 javax.annotation classes.  This is a much simpler solution than that proposed in https://github.com/GoogleCloudPlatform/gcloud-eclipse-tools/pull/261
I see this is a short-to-medium term fix.  I've filed a bug against FindBugs to see about having appropriate modifiers on their OSGi metadata so that just the JSR 305 portion of javax.annotation can be imported.
PTAL @chanseokoh

LGTM

@briandealwis This change should be released to the snapshot to be effective on the Eclipse-side, right?
@meltsufin @loosebazooka Can you push a new snapshot?

Yes. Just did it. Try now.

Thanks! @nbashirbello It's now working great.

Great. Thanks


add option to allow dust from all addresses to be swept up when staking?

I recently discovered that the CLAM consensus doesn't require all the inputs of a staking transaction to be from the same address. The only such rule is that the first input has the same address as the second output (the first output is always zero in a staking transaction).
So it's possible to gather value from multiple addresses while staking. For example:
http://khashier.com/tx/78c1d2cd20bdd13f3497bed2091ccbd0f9811d08766308378166f76498a98578
This may not be desirable for people who want to use the balance at each address as a form of accounting, since the above staking transaction moves coins from one address to another. But for those who don't care about address balances this could be an easy (and free!) way of sweeping up the dust that collects in wallets. (Free, because staking transactions don't pay fees like other transactions do - who would they pay, anyway, other than their own creator).
Currently the client won't combine inputs from different addresses when staking, due to this code in wallet.cpp:
    // we're not splitting the output, so attempt to add more inputs
    BOOST_FOREACH(PAIRTYPE(const CWalletTx*, unsigned int) pcoin, setCoins)
    {
        // Only add coins of the same key/address as kernel
        if ((pcoin.first->vout[pcoin.second].scriptPubKey == scriptPubKeyKernel ||
             pcoin.first->vout[pcoin.second].scriptPubKey == txNew.vout[1].scriptPubKey) &&
            (pcoin.first->GetHash() != txNew.vin[0].prevout.hash ||
             pcoin.second != txNew.vin[0].prevout.n))
        {

Commenting out the first two lines of the if's condition is all that's needed to get the client merging dust from multiple addresses.
I'd propose not changing the default, but having a new config flag: -combineany perhaps, which enables it. (combine, because the flag that currently controls the merging on inputs when staking is called combinelimit)
Thoughts? It would be a trivial change to make, and would allow people (who stake at all regularly) to keep their wallets tidy for free.
I can see an argument against this that we don't want to encourage people to create large transactions without paying for them, but these large transactions are doing a public good - they are reducing the size of the set of unspent outputs - many of which are unspendable otherwise (requiring more in fees to spend than they are worth).

#197 implements this

I'm going to go ahead and pull this to the master branch. Doesn't mean it can't be changed if anyone has any objections.

Here it is in operation, sweeping up unspendable dust outputs for free:
http://khashier.com/tx/1141a4455d2c02788e090c2775c8b5f60c40b32395e37e2e61a6447585598d10

A good feature to have, especially for those whose primary concern is staking together dust.
However, privacy is an additional reason for this to remain off by default.

Fair enough. We do have another solution for cleaning up 'dusty' wallets now (the 500 input limit on automatic coin selection that I implemented a few weeks ago), although that does involve spending more than the dust is worth to clean it up.


New home page changes

I closed other pending PRs to open this one unified PR  these PT stories:

https://www.pivotaltracker.com/story/show/114577553   custom home page content
https://www.pivotaltracker.com/story/show/114576939   custom search path
https://www.pivotaltracker.com/story/show/114578353   custom root for teachers
https://www.pivotaltracker.com/story/show/114578639   home link renamed to 'my classes' (sometimes)

There are some failing tests, and I have to demo this to @scytacki but otherwise this is probably ready for a review.

How to test the new Home Page features in HAS and Learn:
HAS and Learn have two different behaviors for the home page.
HAS wants to show a custom page to anonymous users, but they want that custom page to have the sign-up box, and the regular project header.  In HAS a teacher is redirected to 'getting_started' when they first log in, or when they click on the banner, or visit the root path. In HAS the 'search' button takes the user the full material search page.
In Learn, the custom home page will not show the signup box, or the project header.  It will override all of the portal chrome except for the project footer.  In Learn, the teacher will not be redirected to the 'getting_started' page when they click on the banner (though they will when they first log in). In Learn, the 'search' button will take the user to the custom home page content.  But there will be an 'advanced search' link (linking to '/search/' path) on the custom home page content.
An administrator must configure each portal appropriately.  Here are the settings for each:
HAS:

Add the HAS homepage content to the "Home Page Content" Text area.
The "Wrap Home Page Content" checkbox should be checked (default)
"Custom Teacher Home Path" should redirect to "/getting_started" (default)
The "Custom Search Path" should be blank (default, which will redirect to '/search')

Learn:

Add Ethan's new homepage content to the "Home Page Content" Text area.
The "Wrap Home Page Content" checkbox should NOT be checked
Techers homepage should NOT redirect (undo the default "/getting_started")
Search should link to "/".

This looks good. I'm going to wait for the tests to pass and and then I'll merge it. However we shouldn't deploy it to staging because @eireland is testing learn.staging with the 1.5.0 release.
One thing missing is the ability for a admin to enter in custom getting_started content, we'll have to do that later. Here is a story for it: https://www.pivotaltracker.com/story/show/115074629

These look like the normal random failures.


What is this project?

What exactly is this project? There's no information in the README. All I've been able to find is a couple Ace Editor references to the emmet.js file in this repo.
Is this related to emmetio/emmet? Is this some sort of non-fork from there? Does another (more active) version of this code exist? Or is this the only option for integrating emmet into the Ace Editor?
Specifically, a demo from the official Ace site references the code here: http://ace.c9.io/demo/emmet.html, and I found a few other posts referencing that code which all leads to this mysterious project here.

Thanks!


How to use mapping.txt file to check the original codes?

HI, I am newcomer for android-classyshark,it's easy to use.
But I am confused about how to user mapping.txt to see my original codes.
when the APK file is imported in classyshark,I see the confused name like a.a.a
And I click "file" icon,pulling my mapping.txt to classyshark.
I didn't see any change.

Is my operation correct? Please tell me the right way! Thanks !

@suixiang0923 Have you found a solution for this? I'm also looking for ways to import the mappings into classyshark.

Seems related to #118.


build: allow system-supplied libBlocksRuntime on systems other than Darwin

In the past, the GNUstep project has used the ability so link (a fairly ancient version of) libdispatch against a custom libBlocksRuntime in order to ensure interoperability with its Objective-C runtime (which bundles a BlocksRuntime implementation). It seems that this facility was removed in PR #396 in favour of a check on whether it's building for Darwin to decide whether the in-tree version should be built.
That seems accidental, given the stated purpose of that PR, so I would very much like to re-add that facility. I have now done this by re-adding the elided  find_package(BlocksRuntime QUIET) statement and deciding whether to build the in-tree version depending on BlocksRuntime_FOUND (also the same as prior to PR #396). I hope this is what's intended here. I've checked that this works on a Linux system both with and without a custom libBlocksRuntime.
Unfortunately, I couldn't convince libdispatch to build on my macOS machine – either with or without this change, so that is probably due to my lack of expertise. But it could very well be possible that building the in-tree libBlocksRuntime needs to be conditional on NOT (CMAKE_SYSTEM_NAME STREQUAL Darwin OR BlocksRuntime_FOUND).
Additionally, yet relatedly, there was an extraneous target_include_directories directive in the CMakeLists.txt for the tests/ subdirectory. The effect of that is already achieved by linking the  BlocksRuntime::BlocksRuntime target and it started tripping up things once switching back to find_package() for the runtime lib.
Please let me know what you think!
Thanks,
Niels

Hey folks,
is there any feedback on this? – I'm happy to rework it as needed.
Thanks!
Niels

We’d also like to see this merged – it’d be great if the maintainers could provide feedback. 🙏
Was also wondering if #535 was related?

@compnerd would you mind reviewing this? 🙏

The removal wasn’t really accidental.  The reason is that there’s a number of different blocks runtimes and each has slightly different behaviors in edge cases.  This ensures that all the platform are the same behavior.  I’ll defer to Apple if they want to support different implications, but overall, I think that requiring this version is best.  The rest of the swift build system currently relies on this, so if the external variant is desired to be supported, someone needs to clean up the swift build system first.

The Swift project moved the default branch to main and deleted master branch, so GitHub automatically closed the PR. Please re-create the pull request with main branch.
More detail about the branch update - https://forums.swift.org/t/updating-branch-names/40412


Es-documentation

Updates to README.md that include how to execute the s3-index.exe file.

Added aws configure instructions to the /es/README.md


add2add->add2all and updated docs for extra_add

wrt: https://github.com/fastai/nbdev/issues/137

renamed _add2add to _add2all
updated docs to include detailed use of _all_
hidden the _add2all test cell - not sure it makes sense to see tests for a function that is not in the docs

Thanks! In general though, please don't build the docs when making a PR as it makes the diff hard to read. We prefer to build the docs in separate PRs.


Disable Serialize on other environments

Hi Kam
Is there a way to disable automatic serialisations by config?
My use case:
On my local environment i want that items are being serialised. By using auto synch this is a very easy and good approach.
On my staging environment i don't want that items are being serialised.

I don't want a "Reserialize" option on the unicorn control panel
I want to disable the automatic serialisation to files because its not needed.
I just need "Sync" on the staging environment

One Reason:

The problem is that through automated deployment, the folder where the YML files are being stored, the Unicorn process has no "Write" access and generates an Error in the sitecore content editor.

Is this achievable by config?
Thanks a lot
Regards from Switzerland
Dave

Each config file has a comment as to what type of environments it should be enabled for. In this case you're looking for Unicorn.DataProvider.config to be disabled.
That will not disable reserializing, but will disable writing of updates.

Thanks a lot. I didn't crawl through the comments of the config files, my bad. Perfect! :)


App says no preferred servant or CE is configured when no Autoskill configurations are set

Even if i put the app to choose the first servant it doesn't choose it so i can't start battles automaticly. Also, even if I choose the servant I have to tap the start button because the app doesn't do it.
phone’s model number: SM-A520F
Android version: 7.0
NA
Support Setings:
Ascension settings: ON
Spiritron Costume settings: OFF
Message Display: ON
No blacklisted users

Oh, actually the place where you've placed the play button is where the script checks the screenshot for recognizing support screen. Move it to somewhere else by dragging, I usually put it at bottom-left.

I didn't even notice that the play button is usually included in the screenshot.

Well, now it sends an error message saying "Script Exited: When using preferred suport selection mode, specify at least one servant  or craft essence." but I have put in preferred Waver and Chaldean Lunchtime.

Are you using Autoskill too?
Right now if you're using autoskill, support selection doesn't fallback to the ones set in Support menu but only uses the ones set in Autoskill configuration.
This is a bug and I'll push a fix for this soon.
Unrelated, but, don't use the same mode for Fallback, set it to either Manual or First.

Thanks, now it works perfectly.

Actually, I thinks the app guides more by the craft essence than by the servant, because it actually picked merlin instead of Waver a couple of times.

@josumakina can you share your autoskill configuration entry? I'm talking about the screen with Name, Skill Command and other things on it.

And what are your settings in the "Support" settings screen?

The same that I putted above, I only changed the fallback to "first", the swipes to 4 and the maximun updates to 30.

Hmm, strange. normally the script should ONLY select Waver, but it currently treats your settings like the preferred servant is not set.

Usually misses Merin but there is certain times (not a lot) that chooses it with chaldea lunchtime. Not happened whith any other servant.

@josumakina I just noticed, you're using a Samsung device, right? We'll have a bugfix for that soon with some general info about how to get the app working on Samsung devices.

It choosed Merlin again, and this time I think the app was going to choose the right waver, it missed and choosed Merlin instead, and with 2030.

Yeah, it misses, it almost starts the battle with tamamo now.

We'll be putting up a new build really soon. We've improved the precision of the scrolling code.
Please wait until then.

Can I give you guys a recomendation(an idea) for the scroling? You work is amazing and it will be nice to improve it.

Sure! we'll be happy to know your idea regarding scrolling.
BTW, v0.9.0 is up! Give it a try!
For Android 7, we've reduced the scroll length to half.
For Android 8 and above, we do a scroll and hold gesture.

Ok, this is the idea:

The app istead of touching that space on the bottom left, touches the scroll bar and drags it down scanning the support servants until it reaches the end, when succesfuly finds the servant you want it stops draggind the scroll bar and selects that servant, else, it updates the list. You could input the scrolling velocity on settings.
I think this will be more efective because in the actual mode it touches the servant first for a small moment and then swipes very hard so it ends skipping some servants or missing them when it try to selec them, so when you tap the scroll bar, it doesn't tap any servant at all and the movement of the scroll bar is fixed to the lenght of the support servant list. The scanning metod will be more efective because it won't miss any servant.

I can think of a potential problem with this method.
If you have event filters enabled, the number of servants will be lesser. So, the scroll bar size would change, right? Will it work if we scroll at the same velocity?
Did you give v0.9.0 a try? Does it improve the situation?
Actually, we can do precise scrolls (like your method requires) only on Android 8 and above with the method we're using. On Android 7, it is just a quick swipe.
So, on Android 7 (my phone's also Android 7), we just reduced the amount to scroll by half. This is not the most efficient way to do it, but it seems to work.

I agree with @MathewSachin here, different sizes of the scroll bar would result in different scroll amounts, whereas drag- and dropping has a constant scrolling behavior.

Well, that just a shame for us android 7 users. Also, the problem will be if the scroll bar was smaller, becouse smaller it is, faster it goes, so in events it would be even more precise.
v0.9.0 I think it's a great improvement, I tested it with 5 silver apples and when I came back the job was completed. I'll keep testing and if I see something I'll tell you.

Also, am I or in the past update floating text appeared telling you when it's going to update the support list and the ammount of apples that it have been consumed? Now they are gone, is that intentional?

Seeing it in action the app sometimes fails choosing the noble phantasm card, I think that is because my device is a bit laggy with the game so the app taps the noble phantams card when it is not ready for tapping it. I think that can be fixed with a togleable option called "laggy device" or something like that and what it does is that it extends the timer between tapping the attack button and choosing the cards so the app doesn't pick the card when the game is loading it.

Ok forget what I said above, I have seen the issue in action again, the laggy point is not the command card, is the attack button itself. There is a point that when you use a skill in the game, the game does not let you tap the atack button, so the app taps the untappable attack button and it tryes tapping the cards, and what the app does is open the status info of the servants. If you whant the game to farm alone, thats does nothing because the app completes the quest like always, but if you what a 3 turn farm it the battle extends to 8, 9 turns, so is a big issue.

Regarding "laggy devices", see #27

App avoid the first servant in preferred option, so if the first servant is in the preferred list, the app won't pick it.

@ARFearKubrick Can you test with the latest release, @reconman did some fixes which solved performance issues for most users.
The problem happened on non-720p phones. Your's seems to be 1080p so the fix should improve performance on it.

I tested the lastest release and it's been working well, it picks the noble phantasm card right, I will be seeing the app in action and I will tell you if something odd happens.
Also, you asked for icons in the release of 0.9.0, so I did some. I hope you like it.

We appreacte your designing the icons, but, making icons for Android is not that easy.
For starters, see https://material.io/design/iconography/product-icons.html#design-principles
These things are beyond us and would probably need the help of professional designers.
Also, not to sound rude or anything, but, I'm not a fan of summer Helena's design. Not anything against original Helena though, she's already 10/10/10 Bond 10.


Syntax: define the block of code delimiter

Each programming languages use a different kind of block delimiter, for functions and statements defining:

{} used in C/C++, Java, Go, PHP, C#, Swift, Rust ...
tab and spaces used in Python, Haskell
Special keywords like Begin/End used in Pascal, Delphi, ML
A mix of brace, spaces, and keywords like in Ruby

Chose the Fly solution for maintaining code readability and ease of use.

I think that the use of the braces is consolidated in most of the programming languages used until now.
But I know also that Python which is one of the most popular uses spaces, though there is who prefer Python with brace http://python-with-braces.appspot.com/ and https://github.com/mathialo/bython
Moreover with braces you can define multiple statements in one row without change syntax with additional delimiters:
https://stackoverflow.com/questions/6167127/how-to-put-multiple-statements-in-one-line


fix(slider): support two-way binding for value

Currently we support having a two-way binding via ngModel, but not through the value binding. These changes add support for a two-way binding on the value.

+100 :heart:


fix eloquent generics documentation

[ ] Added or updated tests
[ ] Documented user facing changes

Changes
The documentation mention, we should be using BelongsTo<Model, $this> but when I use it, I got this error (stripped own FQDN)
Method Absence::user() should return
Illuminate\Database\Eloquent\Relations\BelongsTo<User, $this(Absence)> but returns
Illuminate\Database\Eloquent\Relations\BelongsTo<User, Absence>

After changing $this to self it works well.
Breaking changes
None

For me this works:

 /**
  * @return HasMany<BookingDetail, $this>
  */
 public function bookingDetails(): HasMany
 {
     return $this->hasMany(BookingDetail::class, 'bookings_id');
 }


feature(AdminPanel): Quick toggles for Groups/Tests

This commits adds quick switches to enable/disable groups and tests
inside admin management panel
Fixes #392

Anyone want to quickly review?
I need this functionality urgently for 2024.2


fix: make accept necessary work

Currently "accept necessary" does not save the consent given. It should.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


META: All Items that need to be documented

This is a meta issue for all the items that need to be better documented
@MarkPieszak @JohnGalt1717

Information on how to use Transfer Data and Transfer Cache

One thing that was really not obvious that I'd like to write up is the flow for Server and for browser and how they differ from .NET request to server, to return to .NET and output to the page and then the browser pick that up and how it executes through main.browser.ts etc.
A simple diagram of this flow with details of what is getting passed and how would seriously improve the accessibility of this and any project like this.
(I just got transfer_cache working today for the first time and wow, it's great but wow it's completely inaccessible without DEEP knowledge of Angular and SSR other than to maybe blindly use it without understanding how it works.)

@JohnGalt1717 great work. Are the thing you had to change groundbreaking, or just configuration mostly?

@isaac2004 It's (almost) all about Webpack config. IT's just accidental incite based on things others have been saying that puts it all together. Once I get some time I'll do a PR and put it all in.


⚠ Samsung has degraded performance

In 1e12959, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2406 ms

Resolved: Samsung performance has improved in 11c292b.


After uploading entities the user can't see the whole "Data to import" part if the list is long

Problem description
In the Import Data from File modal if the Data import part is moved to the bottom of the page (because of a long list of entities) after uploading entities the user isn’t moved to the bottom of the list/modal – the popup box and the "Append data" button aren't visible so the user doesn't know that so action needs to be taken.
URL of the page
https://test.getodk.cloud/#/projects/385/entity-lists/trees/entities
Steps to reproduce the problem

Navigate to the Entity lists page to the Data tab with a list of at least 50 entities (e.g. the URL above).
Click "Upload”.
Change the number of entities shown per page to 50.
Scroll down so that the "Choose one” button is at the bottom of the page.
Click "choose one” and upload a file with a few entities (or drag the file).

Screenshot
https://jam.dev/c/84fb0f6e-2961-45b1-beb7-c29636511ac2
Expected behavior
After uploading entities the user should see the whole "Data to import" part.
Central version shown in version.txt
https://test.getodk.cloud/
versions:
9a9ec7c034de07e990c9c408ca768326e2154c96 (v2023.5.1-10-g9a9ec7c)
+2f169138bbaecc1b48667a1eb6db40440416dc00 client (v2023.5.0-59-g2f169138)
+60ac8ca5307ff01a151180d5241a8e17a223885e server (v2023.5.0-55-g60ac8ca5)
https://staging.getodk.cloud/
versions:
9a9ec7c034de07e990c9c408ca768326e2154c96 (v2023.5.1-10-g9a9ec7c)
+82438aa97dcd305aac7973736016163a963b95c2 client (v2023.5.0-54-g82438aa9)
+60ac8ca5307ff01a151180d5241a8e17a223885e server (v2023.5.0-55-g60ac8ca5)
Browser
Chrome, Firefox

Tested with Success!

Tested with Success!


switch to shared concurrency

I have run into a throughput problem with the :single concurrency and after a quick read through it seems that using :shared is thread safe as everything in multi_receive creates new memory and I believe the Ruby google api client is already thread safe.
Any objection to making this :shared?

I'm not proficient enough in Ruby to dispute this 😁 I see references to Thread in the Google APIs core, so I agree with you.
Totally happy to merge this.

For peace of mind I have been running this under a heavy test load (200+ thousand log rows a minute into stackdriver without any issues).
Thanks!

That does give me peace of mind - thanks! It took a while, but 0.2.0 of the gem has been published: https://rubygems.org/gems/logstash-output-stackdriver_logging/versions/0.2.0.


[BUG] Can't save text when using Meme - Moerills Expandable Markdown Editor

Describe the bug
When writing a note I can't save it when the markdown editor addon is enabled.
To Reproduce
Steps to reproduce the behavior:

Enable "Meme - Moerills Expandable Markdown Editor"
Open the simple calendar and click "add new note"
Enter a title
Enter some note text
Click save (for the rich text editor)
Click save note
Show the note

Expected behavior
The text to be in the notes field
Browser (please complete the following information):

OS: MacOS
Browser Chrome
Browser Version 89.0.4389.90 (Official Build) (arm64)

Additional context
I'm using the twodsix system (v 0.7.26) and foundry vtt (v 0.7.0)

I personally haven't tried this editor so it will require some investigation to try and make it work.