[Feature Reqeust] Add force_cast(null as integer) support

Feature description:
Now i use the following line to increase null field:
update statis_user_success set cnt=cast(concat('0', cnt) as integer)+1 where key='898234';
Hope:
update statis_user_success set cnt=force_cast(cnt as integer)+1 where key='898234';

Thx for the suggestion.
For your use case there is the coalesce function:
update statis_user_success set cnt = coalesce(cnt, 0) + 1 where key = '898234';

I don't see any advantage that a force_cast function would bring, other than that it might be less to type in certain cases. But it's less flexible than adding a coalesce call and also requires an implicit definition of a default value per type.

I'm using CrateDB Version: 0.56.3,
when "select coalesce(null, 0)+1 from statis_user_success limit 1;",  result: 1,
when "select cnt from statis_user_success limit 1;" result: NULL
when "select coalesce(cnt, 0)+1 from statis_user_success limit 1;",   result: SQLActionException[UnsupportedFeatureException: all arguments for coalesce function must have the same data type]

This happens because cnt is an integer and 0 is interpreted as a long.
coalesce currently does not support mixed arguments (int, long).
See:
cr> select cnt, coalesce(cnt, 0) + 1 as c from statis_user_success limit 1;
SQLActionException[UnsupportedFeatureException: all arguments for coalesce function must have the same data type]
cr> select cnt, coalesce(cnt, cast(0 as integer)) + 1 as c from statis_user_success limit 1;
+------+---+
|  cnt | c |
+------+---+
| NULL | 1 |
+------+---+

We'll improve this behaviour eventually, so that the first example works as well


 You are currently using plaintext http to connect. Please visit the GitHub blog for more information: https://github.blog/2021-08-23-npm-registry-deprecating-tls-1-0-tls-1-1/
163 http fetch POST 426 http://registry.npmjs.org/-/npm/v1/security/advisories/bulk 169ms
164 silly audit bulk request failed Beginning October 4, 2021, all connections to npm websites and the npm registry - including for package installation - must use HTTPS and TLS 1.2 or higher.
164 silly audit
164 silly audit For more information see this blog post: https://github.blog/2021-08-23-npm-registry-deprecating-tls-1-0-tls-1-1/
165 notice Beginning October 4, 2021, all connections to the npm registry - including for package installation - must use TLS 1.2 or higher. You are currently using plaintext http to connect. Please visit the GitHub blog for more information: https://github.blog/2021-08-23-npm-registry-deprecating-tls-1-0-tls-1-1/
166 http fetch POST 426 http://registry.npmjs.org/-/npm/v1/security/audits/quick 25ms
167 verbose audit error HttpErrorGeneral: 426 Upgrade Required - POST http://registry.npmjs.org/-/npm/v1/security/audits/quick
167 verbose audit error     at /usr/local/lib/node_modules/npm/node_modules/npm-registry-fetch/lib/check-response.js:95:15
167 verbose audit error     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
167 verbose audit error     at async [getReport] (/usr/local/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/audit-report.js:335:21)
167 verbose audit error     at async AuditReport.run (/usr/local/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/audit-report.js:106:19) {
167 verbose audit error   headers: [Object: null prototype] {
167 verbose audit error     date: [ 'Wed, 04 Jan 2023 01:15:16 GMT' ],
167 verbose audit error     'content-type': [ 'text/plain' ],
167 verbose audit error     'content-length': [ '271' ],
167 verbose audit error     connection: [ 'keep-alive' ],
167 verbose audit error     'npm-notice': [
167 verbose audit error       'Beginning October 4, 2021, all connections to the npm registry - including for package installation - must use TLS 1.2 or higher. You are currently using plaintext http to connect. Please visit the GitHub blog for more information: https://github.blog/2021-08-23-npm-registry-deprecating-tls-1-0-tls-1-1/'
167 verbose audit error     ],
167 verbose audit error     vary: [ 'Accept-Encoding' ],
167 verbose audit error     server: [ 'cloudflare' ],
167 verbose audit error     'cf-ray': [ '78401f463ed27acb-LAX' ],
167 verbose audit error     'x-fetch-attempts': [ '1' ]
167 verbose audit error   },
167 verbose audit error   statusCode: 426,
167 verbose audit error   code: 'E426',
167 verbose audit error   method: 'POST',
167 verbose audit error   uri: 'http://registry.npmjs.org/-/npm/v1/security/audits/quick',
167 verbose audit error   body: <Buffer 42 65 67 69 6e 6e 69 6e 67 20 4f 63 74 6f 62 65 72 20 34 2c 20 32 30 32 31 2c 20 61 6c 6c 20 63 6f 6e 6e 65 63 74 69 6f 6e 73 20 74 6f 20 6e 70 6d 20 ... 221 more bytes>,
167 verbose audit error   pkgid: undefined
167 verbose audit error }
168 silly audit error Beginning October 4, 2021, all connections to npm websites and the npm registry - including for package installation - must use HTTPS and TLS 1.2 or higher.
168 silly audit error
168 silly audit error For more information see this blog post: https://github.blog/2021-08-23-npm-registry-deprecating-tls-1-0-tls-1-1/
169 timing auditReport:getReport Completed in 233ms
170 silly audit report null
171 timing reify:audit Completed in 233ms
172 timing reifyNode:node_modules/asn1.js-rfc2560 Completed in 232ms
173 timing reifyNode:node_modules/username-sync Completed in 235ms
174 timing reifyNode:node_modules/minimalistic-assert Completed in 237ms
175 timing reifyNode:node_modules/asn1.js-rfc5280 Completed in 237ms
176 timing reifyNode:node_modules/is-stream Completed in 239ms
177 timing reifyNode:node_modules/caseless Completed in 240ms
178 timing reifyNode:node_modules/async-disk-cache Completed in 241ms
179 timing reifyNode:node_modules/mkdirp Completed in 242ms
180 timing reifyNode:node_modules/safer-buffer Completed in 246ms
181 timing reifyNode:node_modules/binaryextensions Completed in 247ms
182 timing reifyNode:node_modules/microsoft-cognitiveservices-speech-sdk/node_modules/https-proxy-agent Completed in 248ms
183 timing reifyNode:node_modules/textextensions Completed in 249ms
184 timing reifyNode:node_modules/microsoft-cognitiveservices-speech-sdk/node_modules/https-proxy-agent/node_modules/agent-base Completed in 250ms
185 timing reifyNode:node_modules/semver Completed in 252ms
186 timing reifyNode:node_modules/bn.js Completed in 250ms
187 timing reifyNode:node_modules/bytesish Completed in 251ms
188 timing reifyNode:node_modules/bent Completed in 252ms
189 timing reifyNode:node_modules/errlop Completed in 254ms
190 timing reifyNode:node_modules/simple-lru-cache Completed in 255ms
191 timing reifyNode:node_modules/istextorbinary Completed in 258ms
192 timing reifyNode:node_modules/heimdalljs Completed in 260ms
193 timing reifyNode:node_modules/editions Completed in 265ms
194 timing reifyNode:node_modules/asn1.js Completed in 268ms
195 timing reifyNode:node_modules/heimdalljs/node_modules/rsvp Completed in 292ms
196 timing reifyNode:node_modules/uuid Completed in 322ms
197 timing reifyNode:node_modules/rsvp Completed in 322ms
198 timing reifyNode:node_modules/microsoft-cognitiveservices-speech-sdk Completed in 1092ms
199 timing reify:unpack Completed in 1094ms
200 timing reify:unretire Completed in 0ms
201 timing build:queue Completed in 3ms
202 info run<EMAIL_ADDRESS>preinstall node_modules/microsoft-cognitiveservices-speech-sdk npm install --package-lock-only --ignore-scripts --no-audit && npx npm-force-resolutions
203 info run<EMAIL_ADDRESS>preinstall { code: 1, signal: null }
204 timing reify:rollback:createSparse Completed in 357ms
205 timing reify:rollback:retireShallow Completed in 2ms
206 timing command:i Completed in 25561ms
207 verbose stack Error: command failed
207 verbose stack     at ChildProcess. (/usr/local/lib/node_modules/npm/node_modules/@npmcli/promise-spawn/lib/index.js:63:27)
207 verbose stack     at ChildProcess.emit (node:events:513:28)
207 verbose stack     at maybeClose (node:internal/child_process:1091:16)
207 verbose stack     at Socket. (node:internal/child_process:449:11)
207 verbose stack     at Socket.emit (node:events:513:28)
207 verbose stack     at Pipe. (node:net:313:12)
208 verbose pkgid<EMAIL_ADDRESS>209 verbose cwd /Users/bob/Projects/flashnews_beta
210 verbose Darwin 21.1.0
211 verbose node v18.12.1
212 verbose npm  v8.19.2
213 error code 1
214 error path /Users/bob/Projects/flashnews_beta/node_modules/microsoft-cognitiveservices-speech-sdk
215 error command failed
216 error command sh -c -- npm install --package-lock-only --ignore-scripts --no-audit && npx npm-force-resolutions
217 error up to date in 13s
217 error
217 error 118 packages are looking for funding
217 error   run npm fund for details
218 error /Users/bob/.npm/npx/73b02210abc194ff/node_modules/npm-force-resolutions/out/npm_force_resolutions/core.cljs:61
218 error   (go
218 error   ^
218 error Error: Timeout trying to fetch resolutions from npm
218 error     at switch__2144__auto (/Users/bob/.npm/_npx/73b02210abc194ff/node_modules/npm-force-resolutions/out/npm_force_resolutions/core.cljs:61:3)
218 error     at /Users/bob/.npm/npx/73b02210abc194ff/node_modules/npm-force-resolutions/out/npm_force_resolutions/core.cljs:61:3
218 error     at npm_force_resolutions$core$find_resolutions$_state_machine__2145__auto____1 (/Users/bob/.npm/_npx/73b02210abc194ff/node_modules/npm-force-resolutions/out/npm_force_resolutions/core.js:476:4)
218 error     at cljs.core.async.impl.ioc-helpers/FN-IDX (/Users/bob/.npm/_npx/73b02210abc194ff/node_modules/npm-force-resolutions/out/npm_force_resolutions/core.js:492:85)
218 error     at cljs.core.async.impl.ioc-helpers/run-state-machine (/Users/bob/.npm/_npx/73b02210abc194ff/node_modules/npm-force-resolutions/out/cljs/core/async/impl/ioc_helpers.cljs:35:23)
218 error     at cljs.core.async.impl.ioc-helpers/run-state-machine-wrapped (/Users/bob/.npm/_npx/73b02210abc194ff/node_modules/npm-force-resolutions/out/cljs/core/async/impl/ioc_helpers.cljs:39:6)
218 error     at fret (/Users/bob/.npm/_npx/73b02210abc194ff/node_modules/npm-force-resolutions/out/cljs/core/async.cljs:490:22)
218 error     at take-cb (/Users/bob/.npm/_npx/73b02210abc194ff/node_modules/npm-force-resolutions/out/cljs/core/async.cljs:188:63)
218 error     at /Users/bob/.npm/_npx/73b02210abc194ff/node_modules/npm-force-resolutions/out/cljs/core/async/impl/channels.cljs:160:45
218 error     at Immediate.cljs$core$async$impl$dispatch$process_messages (/Users/bob/.npm/_npx/73b02210abc194ff/node_modules/npm-force-resolutions/out/cljs/core/async/impl/dispatch.cljs:18:7)
219 verbose exit 1
220 timing npm Completed in 25695ms
221 verbose unfinished npm timer reify<PHONE_NUMBER>036
222 verbose unfinished npm timer reify:build<PHONE_NUMBER>720
223 verbose unfinished npm timer build<PHONE_NUMBER>720
224 verbose unfinished npm timer build:deps<PHONE_NUMBER>721
225 verbose unfinished npm timer build:run:preinstall<PHONE_NUMBER>724
226 verbose unfinished npm timer build:run:preinstall:node_modules/microsoft-cognitiveservices-speech-sdk<PHONE_NUMBER>725
227 verbose code 1
228 error A complete log of this run can be found in:
228 error     /Users/bob/.npm/_logs/2023-01-04T01_15_15_906Z-debug-0.log

Hi @snippet - this looks like an issue on the connection between your client and the npm service. From the error log you posted, I noticed this line
167 verbose audit error 'Beginning October 4, 2021, all connections to the npm registry - including for package installation - must use TLS 1.2 or higher. You are currently using plaintext http to connect. Please visit the GitHub blog for more information:
Here's the blog post mentioned in that error - please let me know if that helps!

closing due to no updates


Add workflow to publish docs

What this PR does

GitHub Actions: Adds a GitHub workflow to publish and deploy docs

Post-merge actions

Merge #8 to get root-level docs published
Once the published site looks as desired, merge cal-itp/eligiblity-server#49

Hmm this was way simpler than I expected it to be. If I had known, I'd have just put it with the docs PR 😁 🤷


Add 'Java. Util. Collections $SingletonMap' to 'org. Springframework. Security. Jackson2. SecurityJackson2Modules' ALLOWLIST_CLASS_NAMES set

I according to the official document "https://docs.spring.io/spring-authorization-server/docs/current/reference/html/guides/how-to-jpa.html" The code that runs gets the correct access_token, then uses the access_token to request the /userinfo endpoint, but gets the following error:
{
    "error_description": "OpenID Connect 1.0 UserInfo Error: The class with java.util.Collections$SingletonMap and name of java.util.Collections$SingletonMap is not in the allowlist. If you believe this class is safe to deserialize, please provide an explicit mapping using Jackson annotations or by providing a Mixin. If the serialization is only done by a trusted source, you can also enable default typing. See https://github.com/spring-projects/spring-security/issues/4370 for details",
    "error": "invalid_request",
    "error_uri": "https://openid.net/specs/openid-connect-core-1_0.html#UserInfoError"
}

I went to consult the https://github.com/spring-projects/spring-security/issues/4370, using the following methods to solve the error:
Create the following two classes：
@JsonTypeInfo(use = JsonTypeInfo.Id.CLASS)
@JsonDeserialize(using = SingletonMapDeserializer.class)
abstract class SingletonMapMixin {

    @JsonCreator
    SingletonMapMixin(Map<?, ?> map) {
    }
}

final class SingletonMapDeserializer extends JsonDeserializer<Map<?, ?>> {

    @Override
    public Map<?, ?> deserialize(JsonParser parser, DeserializationContext context) throws IOException {
        ObjectMapper mapper = (ObjectMapper) parser.getCodec();
        JsonNode mapNode = mapper.readTree(parser);
        Map<String, Object> result = new LinkedHashMap<>();
        if (mapNode != null && mapNode.isObject()) {
            Iterable<Map.Entry<String, JsonNode>> fields = mapNode::fields;
            for (Map.Entry<String, JsonNode> field : fields) {
                result.put(field.getKey(), mapper.readValue(field.getValue().traverse(mapper), Object.class));
            }
        }
        if (result.size() == 1) {
            for (Map.Entry<String, Object> entry : result.entrySet()) {
                return Collections.singletonMap(entry.getKey(), entry.getValue());
            }
        }
        return result;
    }

}

And then configure it：
this.objectMapper.addMixIn(Collections.singletonMap(String.class, Object.class).getClass(),SingletonMapMixin.class);

The above code works, but I'm still confused about the issue. In the "ALLOWLIST_CLASS_NAMES" collection of "org.Springframework.Security.Jackson2.SecurityJackson2Modules", The "java.Util.Collections $SingletonMap" property is not included.
........
static class AllowlistTypeIdResolver implements TypeIdResolver {

		private static final Set<String> ALLOWLIST_CLASS_NAMES;
		static {
			Set<String> names = new HashSet<>();
			names.add("java.util.ArrayList");
			names.add("java.util.Collections$EmptyList");
			names.add("java.util.Collections$EmptyMap");
			names.add("java.util.Collections$UnmodifiableRandomAccessList");
			names.add("java.util.Collections$SingletonList");
			names.add("java.util.Date");
			names.add("java.time.Instant");
			names.add("java.net.URL");
			names.add("java.util.TreeMap");
			names.add("java.util.HashMap");
			names.add("java.util.LinkedHashMap");
			names.add("org.springframework.security.core.context.SecurityContextImpl");
			names.add("java.util.Arrays$ArrayList");
			ALLOWLIST_CLASS_NAMES = Collections.unmodifiableSet(names);
		}
.....

Would it be possible to open-up java.Util.Collections $SingletonMap?  Seen as we already allow java.util.Collections$SingletonList, I can't see adding a java.Util.Collections $SingletonMap would make much different.

@kuschzzp

I according to the official document "https://docs.spring.io/spring-authorization-server/docs/current/reference/html/guides/how-to-jpa.html" The code that runs gets the correct access_token, then uses the access_token to request the /userinfo endpoint

I was not able to reproduce the error. The instructions provided should be more detailed providing step-by-step instructions on how to reproduce.

this.objectMapper.addMixIn(Collections.singletonMap(String.class, Object.class).getClass(),SingletonMapMixin.class)

This is the correct and expected way to register a mixin if the framework does not provide one.

Would it be possible to open-up java.Util.Collections $SingletonMap?

I'm going to close this issue since SecurityJackson2Modules lives in Spring Security and this issue should be logged there.


Update client README - close client resources

The best practice is to close used resources at the end. I've had a problem when I used this code in AWS Golang Lambda function without c.Close(): socket: too many open files - I've reached connection limit because InfluxDB connections have been still opened.

[X] Sign CLA (if not already signed)
[X] Provide example syntax

@jangaraj can you squash the commits for this?

@jsternberg done


[bazel,otp] update manifest with immutable ROM_EXT hash

This PR has two commits that:

Refactor gen-otp-immutable-rom-ext-json.py script to move
the logic for extracting the immutable ROM_EXT section data and
computing the OTP values to a new class ImmutableSectionProcessor
Add a new tool update-manifest-json that updates the manuf_state_creator field in a ROM_EXT manifest with the hash of the immutable ROM_EXT section data.

Note: this depends on https://github.com/lowRISC/opentitan/pull/24948. Only review the last 2 commits.
This PR addresses https://github.com/lowRISC/opentitan/issues/24610 partially.

I rebased on master branch to fix merge conflict. Will merge when this passes CI.

closing as this was submitted as part of #25106


Continue loading tiles when image is not ready yet

When the image of a raster source's source is not available, one of the reasons can be that no tiles are loaded yet. So we need to instruct the tile queue to load more tiles also in this case.
(This problem is another reason why we should rethink or get rid of our tile queue)
Fixes #6279.

Thanks for identifying the issue @fredj and for fixing it @ahocevar.
Please use the "regression" label for any other similar fixes.  I'll cut a patch release with the regression fixes.


Conversion to Enum to allow for specifying custom document types in Item posting

We had a customer request & subsequent product that required we add a new document to BC.
In order to record this document type on the ILE when the document posts a change to inventory, it is required that the 'Document Type' on both the Item ledger Entry & Item Journal tables be converted to an enum.
The tables fields that need to be an enum are:
Table 32 'Item Ledger Entry' - Field 79 'Document Type'
Table 83 'Item Journal Line' -  Field 79 'Document Type'

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


ilab-wrapper: support execution by non-sudoers

DRAFT because I didn't test the Containerfile changes yet.
Also, the way it's currently implemented, I guess there's nothing stopping the user from abusing this script to run pretty much any command.... We're gonna have to think of a way to lock it down so the parameters constructed by the pre-sudo phase are not artificially constructed by the user in such a way that would allow them to execute more than we want them to. Maybe the sudo rule could restrict the first few arguments to be podman run ... instead of *.... No matter which approach I consider, I find very obvious holes...
Background
df8885777d2257a6271ca1a6461673d1d96e99d6
Issue
Running the script requires the user to be a full sudoer, which is not a great requirement to have.
Solution
Create a new group, ilab, and add a sudoers rule that allows users in the ilab group to run the wrapper-script as sudo without a password prompt.
Users will not actually run the script as sudo directly, but the script will re-run itself as sudo.

Decided to look into it in the future, closing for now


Korean long date fix

@jeeeyul , @kyungw00k, @maggiepint , @kyungw00k , please review this pull request when you can.  its to resolve the korean long date formatting issue.  I used to live in Busan, Korea, speak Korean a little bit and felt compelled to pitch in here.  Here is the issue: #3222

@bigtone1284 I think it looks fine. 👍

I was about to make an another PR. I think 1분 represent better than일분. What do you guys think?

@colus001 that isn't the same as this right? Seems like this code is good to go either way.

Actually @bigtone1284 can you get rid of the change to the /locale file? We only want the files in the /src directory modified. If you don't mind rebasing it out that would be appreciated.

I thought it can be part of the pr since this is about korean language. I'll make another pr related to the issue. @maggiepint

@maggiepint I removed my changes from /locale and kept only my changes within /src.

Please rebase this to a single commit.  Thanks.

Please  find the edited pull request at this pull request: #3350

Thanks, but FYI - you don't need to open a new PR, you can just rebase the existing one.  See How to Rebase a Pull Request


breaks with UndoTree

just installed the plugin and it worked just as i wanted, decided to give it some stress and ended up with this
E5108: Error executing lua: function undotree#UndotreeToggle[13]..40[13]..42[19]..<SNR>76_exec[2]..WinEnter Autocommands for "*": Vim(call):E5108: Erro
r executing lua /home/wec/.local/share/nvim/lazy/shade.nvim/lua/shade.lua:182: Invalid window id: 1007
stack traceback:
        [C]: in function 'nvim_win_get_option'
        /home/wec/.local/share/nvim/lazy/shade.nvim/lua/shade.lua:182: in function 'shade_tabpage'
        /home/wec/.local/share/nvim/lazy/shade.nvim/lua/shade.lua:295: in function 'on_win_enter'
        /home/wec/.local/share/nvim/lazy/shade.nvim/lua/shade.lua:436: in function 'fn'
        /home/wec/.local/share/nvim/lazy/shade.nvim/lua/shade.lua:456: in function </home/wec/.local/share/nvim/lazy/shade.nvim/lua/shade.lua:428>
        [C]: at 0x555c9f8efa0c
stack traceback:
        [C]: at 0x555c9f8efa0c

i use u to open it, pressing twice causes an error
al/share/nvim/lazy/nui.nvim/lua/nui/split/init.lua:170: WinEnter Autocommands for "*": Vim(call):E5108: Error executing lua /home/wec/.local/share/nvim
/lazy/shade.nvim/lua/shade.lua:182: Invalid window id: 1007
stack traceback:
^I[C]: in function 'nvim_win_get_option'
^I/home/wec/.local/share/nvim/lazy/shade.nvim/lua/shade.lua:182: in function 'shade_tabpage'
^I/home/wec/.local/share/nvim/lazy/shade.nvim/lua/shade.lua:295: in function 'on_win_enter'
^I/home/wec/.local/share/nvim/lazy/shade.nvim/lua/shade.lua:436: in function 'fn'
^I/home/wec/.local/share/nvim/lazy/shade.nvim/lua/shade.lua:456: in function </home/wec/.local/share/nvim/lazy/shade.nvim/lua/shade.lua:428>
^I[C]: in function 'nvim_command'
^I...c/.local/share/nvim/lazy/nui.nvim/lua/nui/split/init.lua:170: in function <...c/.local/share/nvim/lazy/nui.nvim/lua/nui/split/init.lua:169>
^I[C]: in function 'nvim_win_call'
^I...c/.local/share/nvim/lazy/nui.nvim/lua/nui/split/init.lua:169: in function '_open_window'
^I...c/.local/share/nvim/lazy/nui.nvim/lua/nui/split/init.lua:257: in function 'mount'
^I...are/nvim/lazy/neo-tree.nvim/lua/neo-tree/ui/renderer.lua:1018: in function 'acquire_window'
^I...
^I...y/neo-tree.nvim/lua/neo-tree/sources/filesystem/init.lua:171: in function <...y/neo-tree.nvim/lua/neo-tree/sources/filesystem/init.lua:170>
^I[C]: in function 'pcall'
^I...hare/nvim/lazy/neo-tree.nvim/lua/neo-tree/utils/init.lua:126: in function 'debounce'
^I...y/neo-tree.nvim/lua/neo-tree/sources/filesystem/init.lua:170: in function 'navigate'
^I...nvim/lazy/neo-tree.nvim/lua/neo-tree/sources/manager.lua:493: in function 'navigate'
^I...re/nvim/lazy/neo-tree.nvim/lua/neo-tree/command/init.lua:204: in function 'do_show_or_focus'
^I...re/nvim/lazy/neo-tree.nvim/lua/neo-tree/command/init.lua:162: in function 'execute'
^I...re/nvim/lazy/neo-tree.nvim/lua/neo-tree/command/init.lua:170: in function '_command'
^I[string ":lua"]:1: in main chunk
^I[C]: at 0x555c9f8efa0c
stack traceback:
^I[C]: in function 'nvim_command'
^I...c/.local/share/nvim/lazy/nui.nvim/lua/nui/split/init.lua:170: in function <...c/.local/share/nvim/lazy/nui.nvim/lua/nui/split/init.lua:169>
^I[C]: in function 'nvim_win_call'
^I...c/.local/share/nvim/lazy/nui.nvim/lua/nui/split/init.lua:169: in function '_open_window'
^I...c/.local/share/nvim/lazy/nui.nvim/lua/nui/split/init.lua:257: in function 'mount'
^I...are/nvim/lazy/neo-tree.nvim/lua/neo-tree/ui/renderer.lua:1018: in function 'acquire_window'
^I...ree.nvim/lua/neo-tree/sources/filesystem/lib/fs_scan.lua:546: in function 'get_items'
^I...y/neo-tree.nvim/lua/neo-tree/sources/filesystem/init.lua:150: in function '_navigate_internal'
^I...y/neo-tree.nvim/lua/neo-tree/sources/filesystem/init.lua:171: in function <...y/neo-tree.nvim/lua/neo-tree/sources/filesystem/init.lua:170>
^I[C]: in function 'pcall'
^I...hare/nvim/lazy/neo-tree.nvim/lua/neo-tree/utils/init.lua:126: in function 'debounce'
^I...y/neo-tree.nvim/lua/neo-tree/sources/filesystem/init.lua:170: in function 'navigate'
^I...nvim/lazy/neo-tree.nvim/lua/neo-tree/sources/manager.lua:493: in function 'navigate'
^I...re/nvim/lazy/neo-tree.nvim/lua/neo-tree/command/init.lua:204: in function 'do_show_or_focus'
^I...re/nvim/lazy/neo-tree.nvim/lua/neo-tree/command/init.lua:162: in function 'execute'
^I...re/nvim/lazy/neo-tree.nvim/lua/neo-tree/command/init.lua:170: in function '_command'
^I[string ":lua"]:1: in main chunk
^I[C]: at 0x555c9f8efa0c

and all this if i tried opening neotree with e

skill issue tbh

skill issue tbh

get banned so we can all laugh about it next friday


Add moderation guide

@VirtuallyThere @batcavekid Added a moderation guide, please take a look and let me know if it looks good! I try to do one additional review for all PRs, so feel free to ack so I can merge.

Looks good to me as an initial guide. Are there any other config options that should be documented?

Awesome, thanks for reviewing! The only config option right now is hidden. It was an emergency patch once the first butthole appeared. Didn't know the ordinals logo would be so prophetic ◉


Set MongoDB credentials for auth & frontend

In https://github.com/openstad/openstad-frontend/pull/354 and https://github.com/openstad/openstad-oauth2-server/pull/112 new environment variables have been created to allow the auth & frontend services to connect to MongoDB with authentication.
This PR allows the k8s environment to make use of these new variables through secrets (fed by values).
There were a few small issues with linting that are also being resolved in this PR to allow for helm lint to pass succesfully.

Drafting this PR for now — I will test this k8s setup before marking it as Ready to review.

I have just finished testing this PR (with images for https://github.com/openstad/openstad-frontend/pull/354 and https://github.com/openstad/openstad-oauth2-server/pull/112) on our test cluster, and everything is working as intended!


Sorting of system log inverted after refresh

If you go to 'Status und Bedienung' / 'Systemprotokoll' you normally get a date sorted list with increasing datestamps. If you click on 'Aktualisieren' (the button, not in browser), the list reloads, but is inverted, starting with the newest datestamp and decreasing.
Tested it with Crome and Firefox.

This doesn't seem to be a RaspberryMatic specific problem but will certainly also be a problem with a plain CCU2.

fixed in 3.53.26 ?
[HMCCU-645]Die Sortierreihenfolge imSystemprotokoll wechselte bei jeder Aktualisierung der Daten.

should be indeed fixed with 3.53.x. Thanks for the hint.


Reapply the OptimizedModule in Fabric.setup()

What does this PR do?
Part of #17250
When you do
model = torch.compile(orig_model)
model = fabric.setup(model)

today, then you end up with
FabricModule(StrategyWrapper(OptimizedModule(orig_model)))

where StrategyWrapper represents the wrapper applied by the strategy, e.g., FSDP, DDP, DeepSpeedEngine, etc.
With the new feature enabled in this PR, you will now get:
FabricModule(OptimizedModule(StrategyWrapper(orig_model)))

Note how the optimized module gets applied outside the strategy wrapper, but inside the fabric module router.
It is important to have the fabric module outside the optimized module, because it contains only convenience routing and precision management that does not need to be compiled (nor can it be without graph breaks atm). At the same time, the FSDP/DDP specific operations can still be captured properly by torch compile, and how it is intended to be wrapped in PyTorch.
This feature is marked as experimental because we first need to test its application in the wild.
Follow-up work:

Verify it in the wild
Decide on the default
Document it in the compile-guide
Do the same in Lightning Trainer

📚 Documentation preview 📚: https://pytorch-lightning--19192.org.readthedocs.build/en/19192/

cc @borda @carmocca @justusschock @awaelchli

Closing in favor of alternative approach #19280


Could please someone push fix

In my last issue I provided error, regarded to mc_cursor https://github.com/comtihon/mongodb-erlang/issues/200. Could please some fix it - add to mc_worker.erl one clause
get_database(#getmore{}, ConnState) -> ConnState#conn_state.database;

Hi, sorry for long delay.
I am on sick leave now. Can you please create a PR (with minor version increment)? I will merge it.


Post on Cyber Security Capability Maturity Model

Tracked with this branch 27CyberMaturityModel

also

https://dodcio.defense.gov/CMMC/Model/


impl PhfHash for (...)

Out of curiosity, what are you using tuple keys for?

Map from (key code, modifier) to action in a terminal emulator

Does a match not optimize well for that use case?

It might, but i'm generating the table from a static config file and this just seemed easier.

Cool thanks for the info. For some background, I'm thinking about adjusting the API from a Hash based one to one that just passes a byte buffer. This will provide some more flexibility to look into approaches like gperf's, where it constructs a custom hash function that only looks at a minimal number of bytes required to make the hash unique. Unfortunately, this doesn't really work with composite types like tuples in a super easy way.


clpz: expands unrelated terms

$ /opt/gupu/new-trealla/tpl -f
?- use_module(library(clpz)).
   true.
?- [user].
g_arithrel(#=(A1,A2), A1, A2).

   true.
?- g_arithrel(R,A1,A2).
   R = (integer(A1)->(true,var(A2)->A2 is A1;integer(A2)->A1=:=A2;_A is A1,clpz:clpz_equal(_A,A2));integer(A2)->(true,var(A1)->A1 is A2;_A is A2,clpz:clpz_equal(A1,_A));clpz:clpz_equal(A1,A2)),
         unexpected.
   R = (A1 #= A2). % expected, but not found

And there is no reason to expand it here:
?- [user].
q(X,X).
   true.
?- q((A1#=A2),R).
   R = (integer(A1)->(true,var(A2)->A2 is A1;integer(A2)->A1=:=A2;_18 is A1,clpz:clpz_equal(_18,A2));integer(A2)->(true,var(A1)->A1 is A2;_18 is A2,clpz:clpz_equal(A1,_18));clpz:clpz_equal(A1,A2)).

Expansion can only be done at term-to-body conversion time just for goals, not other arguments.

SICStus docs:

Only goals appearing in the body of clauses when reading a source file are expanded using this mechanism, and only if they appear literally in the clause, or as an argument to a defined meta-predicate that is annotated using‘0' (see meta_predicate/1). Other cases need a real predicate definition.

It seems I made Trealla too lax here as it's not checking for '0' annotation only.

Note: clpz.pl uses put_attr/3 goal expansion with member/2 on arg2.

Just a rule-of-thumb: The expansion as such must remain invisible except for clause/2.

And indeed meta-arguments.

For meta-arguments that represent a single goal (that is, that do not lack further arguments), it is OK. But for 1, 2, ... meta-arguments I'd rather wait until call/N time, if at all.


vue i18n warnings

OS, browser, uiv version
elementary os (ubuntu 16.04), chrome, uiv version 0.11.10
Which component
modal
Steps to reproduce the problem
use modal component with nuxt and vue i18n plugin and in terminal and in browser console show this warnings:
[vue-i18n] Value of key 'uiv.modal.cancel' is not a string!
[vue-i18n] Value of key 'uiv.modal.cancel' is not a string!
[vue-i18n] Cannot translate the value of keypath 'uiv.modal.cancel'. Use the value of keypath as default.
[vue-i18n] Value of key 'uiv.modal.ok' is not a string!
[vue-i18n] Value of key 'uiv.modal.ok' is not a string!
[vue-i18n] Cannot translate the value of keypath 'uiv.modal.ok'. Use the value of keypath as default.
component template:
<template lang="pug">
  modal(v-model="isOpen")
    cm-header-modal(slot="header", :title="title", @close="close")

    cm-login-form(v-show="form === 1", :reset-data="isOpen", @change-form="changeForm", @success="close")
    cm-reset-password-form(v-show="form === 2", :reset-data="isOpen", @change-form="changeForm")
    cm-register(v-show="form === 3", :reset-data="isOpen", @change-form="changeForm", @success="close")
    cm-register-form(v-show="form === 4", :reset-data="isOpen", @success="close")

    template(slot="footer")
      .row
        .col-xs-12
          span.text-center.legal-login Test
</template>

Expected behavior
no warnings

Have you merge the language pack of this lib to your app's?

that was the problem, my app use spanish lang and in uiv is not defined. thank you!


fix(aya-sock-map): invalid transmute when calling fd

Corrent an invalid transmutation for sock_map.
fd is already a ref of MapFd, so transmuting &fd to &SockMapFd is equivalent to transmuting &&SockMapFd into &SockMapFd which is buggy.
Step to reproduce
This simple program terminates in error with Os error 9, invalid file descriptor
    let mut intercept_ingress: SockMap<_> = bpf.take_map("INTERCEPT_INGRESS").unwrap().try_into()?;
    let map_fd = intercept_ingress.fd().try_clone()?;

SockHash is doing it correctly BTW https://github.com/aya-rs/aya/blob/main/aya/src/maps/sock/sock_hash.rs#L113

You welcome, thanks for aya :)


1.10.2

1.10.2

Compatible with game version 1.201.012
Removed the turret OnEndShoot crash fix (Keen fixed it)

1.10.2

Compatible with game version 1.201.012
Removed the turret OnEndShoot crash fix (Keen fixed it)


Add extra note to readme

This one caught me out, it was odd because the fields were marked as not being valid, yet ctrl+click worked - turned out my endpoints url was incorrect which caused it.
Quick question as well, is it possible to for specific projects only enable syntax highlighting from this plugin?

Hi Matthew.
Thanks for taking the time to contribute.
The endpoint url is used only for query execution, e.g. in .graphql (scratch) files. Only the "file" and "request" fields can affect the loaded schema. But note that if you use "request", and saved changes to graphql.config.json, e.g. the endpoints url, it will trigger a re-fetch of the schema.
Can you elaborate, because I don't see this change being applied as is.
Thanks,
Jim.

To answer your question, no the schema-based error annotator runs automatically as part of the plugin.

Hey @jimkyndemeyer So this was erroring when using the Relay.QL tagged syntax.
Things like the viewer query were being marked as invalid fields.
This corrected itself once I updated the endpoint url. Does the plugin use this url for the introspection query?
I had actually refreshed the schema from the errors tab numerous times before I discovered this.
If you're still not following I'll put together a quick screencast in the next couple of days.
--
Matt

I'll try to outline the specific URLs so we're sure we understand each other.
The introspection query is controlled by the following fields: https://github.com/jimkyndemeyer/js-graphql-intellij-plugin/blob/master/resources/META-INF/graphql.config.json#L10-L14
The plugin also allows you to execute other relevant queries against your endpoint, e.g. for testing, much like you would a traditional database. This is controlled by https://github.com/jimkyndemeyer/js-graphql-intellij-plugin/blob/master/resources/META-INF/graphql.config.json#L26-L29
I think that what you're seeing is a result of modifying the config file, which then triggers the introspection query. The introspection query will also be trigger by the "Restart ... Language Service" action.

Closing this based on the 2.0 release and the new documentation website at https://jimkyndemeyer.github.io/js-graphql-intellij-plugin/


Subfolders under collection's folder causes "Loading Entries" indefinitely (Github backend)

Disclaimer: I'm very new to netlify-cms (started yesterday with about 6 hours of playing with it) and may be completely off on this, but it seems to consistently happen.
- Do you want to request a feature or report a bug?
Bug most likely
- What is the current behavior?
With a github backend, having a sub-folder in the folder that is a collection folder target causes the collection's entries to never load.
I.e. I am adapting an existing static site setup to used the cms. I have the folder src/documents/posts. The files in my src/documents folder are my "main pages" and the sub-folders under that are related to each main page. I have one collection for posts that loads fine, but my "main pages" collection does not load. If I create a sub-directory under src/documents/posts then my posts collection no longer loads.
- If the current behavior is a bug, please provide the steps to reproduce.
Use Github backend. Have a collection pointing to a folder. Have a sub-folder in that collection's folder.
- What is the expected behavior?
The collections should ignore folders and only deal with files.
- Please mention your node.js, and operating system version.
version v7.7.3, I'm using netlify for the testing.

As for fixing it I tried to track down the place in the code that should maybe change. https://github.com/netlify/netlify-cms/blob/master/src/backends/github/API.js#L155 the ListFiles could have a for loop that strips out folders from the JSON pulled from github since it is only supposed to list files according to it's name not folders also.
Example: https://api.github.com/repositories/85621207/contents/ shows that the json has a field called "type" which is "file" or "dir". The for loop would look for and remove anything not having the file type.
As i said at the start I am new to netlify-cms so this may not be the actual issue or the right way to fix it. I do know that when a collection has sub-folders it loads entries indefinitely and when I remove those subfolders it loads fine. I tweaked my configuration so my main pages were in their own folder and it fixed the issue, but is causing me some other problems since I have to move those files to the right place afterwards (for the quick fix I used this hacky function to move main pages where they need to be).
Some extra information:

I am using metalsmith to generate the site
My testing repository is https://github.com/DomtronVox/StaticFarmSite/
I have edited the repository since I had the bug to work around it. commit 07caa6f11dbbf9589917912b861011b7b220c30b is where I was having the issue.

Thank you, btw I am really liking the project so far.

Seems related to #218

Fixed by #370


Redirect not working

I'm trying to use this gem to enforce the SSL on the site. If I got to mydomain.com, it correctly redirects me to https://www.mydomain.com. Once I'm there, if I take off the www. and just go to http://mydomain.com, it renders the site at http://mydomain.com, which breaks the SSL since the SSL only works with the WWW subdomain.
Here's what I have in my production.rb file:
config.middleware.use Rack::SslEnforcer, :redirect_to => 'https://www.mydomain.com', :strict => true
Any thoughts on how I can force the site to always render https://www.mydomain.com?
Thanks!

If you don't want to force http for any paths / subdomains / whatever,
just remove the strict option.
Best,
Tobi

Tobi-
Thanks for your reply.
I do want to enforce SSL at all times. Only the "www." sub-domain is setup to work SSL'd. Initially going to the domain, the gem redirects to https://www.mydomain.com, but once I'm there, if someone takes off the "www." and just goes to https://mydomain.com, it doesn't redirect you back to https://www.mydomain.com, as it should.
Matt
On Mar 7, 2012, at 12:27 PM, Tobias Matthies wrote:

If you don't want to force http for any paths / subdomains / whatever,
just remove the strict option.
Best,
Tobi

Reply to this email directly or view it on GitHub:
https://github.com/tobmatth/rack-ssl-enforcer/issues/34#issuecomment-4372860


feat(scripts): created scripts directory and populated it

compute_bounds.py: given a layer, compute the bbox and chunk size so we can set info accurately
generate_manifests.py: generate manifests for a newly meshed object. Can also be done with tasks.
ngmesh2obj.py: convert a neuroglancer mesh into .obj representation. useful for sharing outside the lab.
remap2npy.py: convert an .h5 segmentation remap into a numpy representation used by the pipeline
super_downsample.py: for small volumes, you can superdownsample using this script, but it is generally supersceded by tasks.
Added documentation to scripts

This is a revival of #130

Here's the mesh documentation: https://github.com/google/neuroglancer/tree/master/src/neuroglancer/datasource/precomputed#mesh-representation-of-segmented-object-surfaces

LGTM


[$1000] Chat comment linking redirects to 'Hmm... it's not here' instead of the intended chat

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
beta flag: commentLinking

Open the chat.
Copy link to the message
Put this link in the chat
Click on this link.

Expected Result:
The application should redirect to the referenced chat.
Actual Result:
Instead of navigating to the intended chat, the application returns a 'Hmm... it's not here' page.
Workaround:
do not need as the comment linking feature is in development
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.46-0
**Reproducible in staging?:**y
**Reproducible in production?:**y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @perunt
Slack conversation:
https://github.com/Expensify/App/assets/48593211/b855e3bb-f460-4fbe-ad8e-345ac3c8ebb1
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01aef64f5f78bfffbd
Upwork Job ID:<PHONE_NUMBER>680666112
Last Price Increase: 2023-08-04

Use beta flag 'commentLinking' to enable the possibility of 'copy link to message'
https://github.com/Expensify/App/assets/48593211/dcd3a7da-12f0-4f99-add8-74913b2f1bc7

Proposal
Please re-state the problem that we are trying to solve in this issue.
For further implementation of the comment linking feature, we need this functionality of navigating to chat works. Now we have error page instead of redirection to the proper report room
What is the root cause of that problem?
Navigator link formatting
What changes do you think we should make in order to solve the problem?
we need update this part of the code
and put
REPORT_WITH_ID: 'r/:reportID/:reportActionID?',

also expand propTypes here as well
/** Navigation route context info provided by react navigation */
route: PropTypes.shape({
    /** Route specific parameters used on this screen */
    params: PropTypes.shape({
        /** The ID of the report this screen should display */
        reportID: PropTypes.string,

        /** The reportActionID to scroll to */
        reportActionID: PropTypes.string,
    }).isRequired,
}).isRequired,

@alexpensify FWIW I asked @perunt to create this issue

@dukenv0307, it seems my solution found its way into your edited comment 🤣
Anyway, in the scope of this task, I would also fix the format of the link created, as now it needs to be revised. Can I do it in this scope, or should I create a new issue?

@roryabraham thanks for the update.
I've assigned external. @mananjadhav when you get a chance please review the proposals. Thanks!

@roryabraham do we have any preference on the URL? Path params vs query params. I am inclined to use @perunt's proposal as it is inline with rest of our URLs, but do we need any kind of identifier between the reportId and reportActionId?

@mananjadhav it's preparing to this task.

@roryabraham when you get a chance, can you review the comments starting here: https://github.com/Expensify/App/issues/23711#issuecomment-1656336329

@cead22 - Rory is OOO until next week. To keep this moving forward, we need feedback on this question: https://github.com/Expensify/App/issues/23711#issuecomment-1656336329
Thank you!

Carlos is OOO now, so I'm going to hold for now. I think Rory is back on Monday.

Update: we need a review here before we can move forward

Hey, sorry for any confusion here but @perunt was hired separately to work on the comment linking project and wrote a design doc which was reviewed by our team, so this really wasn't meant to be open to external proposals.
Regarding @mananjadhav's question, we're using path params, not query params. The format should be: /r/:reportID/:reportActionID?

Thank you for the update, Rory!

@roryabraham, I don't want to distract you, but I also want to create an issue regarding the formatting of the copy link. I recall a conversation about it in the comment linking document, but the issue persists. I'm going to fix it before your PRs are ready.

@roryabraham and @mananjadhav - since there are various parts to this one, should we put on Hold?

No update

Weekly Update: On Hold

@mananjadhav to prepare for the payment day, can you please complete the checklist? Thanks!

@mananjadhav - payment day is tomorrow, can yo please complete the checklist? Thanks!

@alexpensify thanks for following up. I would treat this as a part of feature request, where we land on the specific action after clicking the link. Here we're only ensuring that it lands to a specific chat instead of Hmm... it's not here.
Hence I think this doesn't need a regression test too.
@alexpensify Can you post a payout summary?

Here is the payment summary:

External issue reporter @perunt $250
Contributor that fixed the issue @perunt $1000
Contributor+ that helped on the issue and/or PR @mananjadhav $1000

Upwork Job: https://www.upwork.com/jobs/~01aef64f5f78bfffbd
*If applicable, the bonuses will be applied on the final payment
Extra Notes regarding payment: There is no bonus and no penalty, this GH required multiple steps and reviews.

@mananjadhav - let me know if there are questions or concerns, I will close tomorrow. Thanks!

$1,000 payment approved via NewDot based on BZ summary.


Monthly Mean data ingestion

Adding data from files in monthly-mean Azure blob container to Postgres tables
Tasks:

[ ] Create container field and methods to upsert data according to this field
[x] Create SimulationMonthlyMeanOutput to host relevant data from netCDF files
[x] Change SimulationSample.js afterCreate to create the correct files every time a new SimulationSample is added

I added a container field to the SimulationOutputFile type that now is:
entity type SimulationOutputFile schema name 'SMLTN_OTPT_FL' {
    // The {@link SimulationSample} that theses files belong to.
    simulationSample: !SimulationSample
    // The date for this file (comes with the file name)
    dateTag: datetime
    // The simulation output file
    file: !File
    // The container this file belongs to
    container: !string enum('acure-aircraft', 'monthly-mean')
    // processed
    processed: boolean post default "false"

    // Load data from this file into {@link SimulationModelOutput}
    @py(env='gordon_1_0_0')
    upsertAcureAircraftData: member function(): boolean
    // Load data from this file into {@link SimulationMonthlyMeanOutput}
    @py(env='gordon_1_0_0')
    upsertMonthlyMeanData: member function(): boolean
    // Load data from all containers
    @py(env='gordon_1_0_0')
    upsertData: member function(): boolean
}

The methods will do different things according to this field.

Here's how the Simulation types relate at this point:

Ideally we would change SimulationModelOutput to SimulationAcureAircraftOutput, but that would trigger a cascade of conflicts in notebooks and other types that I'm not sure is worth dealing with...

I created a fake SimulationSample by adding an extra row to the .csv Source file, and apparently the container field is working, but filtering the correct SimulationSample is not...

The new afterCreate callback seems to be working fine. Now to the Python methods....

Ok, so now I'll delete from Postgress the last two SimulationSamples (the very last one is fake and needs to be gone anyway), reprovision the entire package (which should trigger SimulationSample.afterCreate()), and then try to upsert the new SimulationMonthlyMeanOutput table. Fingers crossed!

After provisioning, the new files seem to be there!

Now let's test the python method....

From the console...
var file = SimulationOutputFile.get('11617390-95f4-489d-a3b7-3b0d2d6ade67')
file.upsertMonthlyMeanData()

That took a few minutes... and failed.

Not a lot of info here...

It worked! After 3.5min, I upserted the data from one single file:

Next step would be cleaning the SimulationSample table, reprovision the package with reset option, then triggering the BatchJob to upsert all new data

Methods are all working. Just need to do a @db unique=["time", "latitude", "longitude"] on SimulationMontlyMeanOutput to avoid duplicates

Regarding the Azure Blob Storage thing... For some reason, when I cached all the files, I was able to see other folders that I could not see before. In fact, there's exactly one folder for each month with about 5,000 files in each of them. I'm gonna change the code to go into each one.
I also found out why I only got 22 files for a single simulation... FileSystem.inst().listFiles() has a default limit of exactly 5,000. Since we have 221 simulations, which is basically the stride to find the next file for a certain SimulationSample, 22x221 = 4862, so the next file would be out of range.... A better approach is to use the listFilesStream instead, checking if the returned object hasNext() at each iteration. Let me try that.

The solution to find the files for a certain SimulationSample object is surprisingly a bit more elegant than what we had before:
    var simString = padStart(String(obj.simulationNumber), 3, '0');
    var sampleFiles2 = new Array();

    const months = ['jan', 'feb', 'mar', 'apr', 'may', 'jun',
                    'jul', 'aug', 'sep', 'oct', 'nov', 'dec'];
    var containerRoot = "azure://monthly-mean-simulations/";

    for (var month in months) {
      var pathToFiles = containerRoot + month + "/";
      var fileStream = FileSystem.inst().listFilesStream(pathToFiles);
      while (fileStream.hasNext()) {
        var file = fileStream.next();
        if (file.url.slice(-6,-3) === simString && file.url.slice(-3) === ".nc" && file.url.slice(37,42) !== 'ACURE') {
          sampleFiles2.push(file);
        }
      }
    }


Add [Phishing] #malicious GDXXOO…MSMBOT

Address: GDXXOO7A2OJTZGGZTYNVYHN4FFLL7E6N72GUOLB5ZDVWEMKXNMMSMBOT
Tags: malicious
Title: Phishing
Requested by @sysmanalex

Related to scams rippie.claims / eurb.net / dstoq.tech / flare-finance.org

How are they related to these known scams? Can you please give more details about this address as to its fraudulent activities. Can you supply transaction numbers or screen shots? A chain of transactions is the most helpful. Do they have a website?

@sysmanalex can you please give some more detail about these connections you have seen?
rippie.claims / eurb.net / dstoq.techare out right scams infringing on Trademarks, Brands, and Logos. flare-finance.org comes close by saying that their various tokens (no trademark or branding issues) are "Institutional-Grade Finance Built On The Flare Network" but that can mean their own "network since they do not steal the logo or claim a different domain. Their flare-finance.org is just a dead domain.
I agree that these also are most likely fraudulent, but flagging is not done based on opinion, only the facts.


Tapping the already active tab should scroll content up to top

Many apps (see Twitter for an example) scroll to the top of the content when you tap the active tab.
Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x ] 3.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[x ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

Tapping the active tab does nothing
Expected behavior:

Tapping the active tab would scroll to the top of the content
Steps to reproduce:

Related code:
insert any relevant code here

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
insert the output from ionic info here

Interesting. Is this default behaviour or only chosen behaviour of some apps?
Is there a way to achieve this manually?

I have done this by:

Including 'Content' ionic dependency in the page.
Having the following code:

  // When this tab is selected
  ionSelected() {
    this.content.scrollToTop(); // 300ms
  }

@fephil Can you add your ionic info output to your issue so we can see which version you exactly use?

@danielsogl As requested:
global packages:

    @ionic/cli-utils : 1.3.0
    Cordova CLI      : 6.5.0
    Ionic CLI        : 3.3.0

local packages:

    @ionic/app-scripts              : 1.3.7
    @ionic/cli-plugin-cordova       : 1.3.0
    @ionic/cli-plugin-ionic-angular : 1.3.0
    Cordova Platforms               : android 6.2.3 ios 4.4.0
    Ionic Framework                 : ionic-angular 3.3.0

System:

    Node       : v6.11.0
    OS         : macOS Sierra
    Xcode      : Xcode 8.3.3 Build version 8E3004b
    ios-deploy : 1.9.1
    ios-sim    : 5.1.0

@fephil when switching tabs does this scrollToTop? The behavior I'm looking for would only scroll to top if the current tab is already active

@piotrowski not 100% sure if it's default behavior. But many/most apps do this: Facebook, Twitter, LinkedIn. Enough to make me think "this is odd" when tapping the selected tab did not scroll me to the top.

@joeflateau no, it works as I believe you want it to work.

@fephil nice! I'm trying now...

@fephil wonder too if this could be made into a class decorator to simplify things.

@fephil sort of works. but if the content has some scroll momentum already it gets janky

Interesting, I don't get that behaviour on my app/devices. Perhaps adjust the animation time?

@fephil I'll bet it's because I also have pull to refresh on that page, tbh

@fephil nope, removed the refresher and same behavior. if it's already scrolling, the "scrollToTop" animation happens but then the page jumps to the place where the previous momentum would have it end up.

@joeflateau Sorry, can't help as I don't get that behaviour (just checked again).

Workaround for this of course would be to "stop" or "cancel" the active motion somehow.

Hello everyone! Thanks for the feature request. I'm going to move this issue over to our internal list of feature requests for evaluation. We are continually prioritizing all requests that we receive with outstanding issues. We are extremely grateful for your feedback, but it may not always be our next priority. I'll copy the issue back to this repository when we have begun implementation. Thanks!

This issue was moved to ionic-team/ionic-feature-requests#252

@jgw96 @fephil sorry to re-open this, i just realized that tapping the status bar to scroll to top works as expected (if a scroll is in progress it's aborted and the content is scrolled to top.) I am having trouble finding the code that actually does that, any ideas?

@joeflateau I think you are looking for https://github.com/ionic-team/ionic/blob/a8ceee467b1d83a74d875e2162732c31d72c566e/src/components/tabs/tabs.ts#L556-L559


Typescript Library - argbFromHex - Incorrect Parsing

The argbFromHex function does not parse hex values correctly when the hex value is 8 characters long (excluding the #).  It incorrectly assumes the last two characters are the blue channel when it's actually the alpha channel.
Example: 50% opacity as represented by the 80 in the input (black)
Input: #00000080 (equivalent to #7f7f7f)
Incorrect Output:<PHONE_NUMBER> (equivalent to #000080) ... a blue
https://github.com/material-foundation/material-color-utilities/blob/9e67d5efa68a522835d8975064a95000f1f8e1d5/typescript/utils/string_utils.ts#L50-L78

I know you guys aren't accepting pull requests....but it looks like alpha isn't being factored in a few parts of the code base.  The following is a working version of the function in question (with minimal refactoring):
export const argbFromHex = (hex: string) => {
  hex = hex.replace('#', '');
  const isThree = hex.length === 3;
  const isSix = hex.length === 6;
  const isEight = hex.length === 8;
  if (!isThree && !isSix && !isEight) {
    throw new Error('unexpected hex ' + hex);
  }
  let r = 0;
  let g = 0;
  let b = 0;
  let a = 0;

  switch (hex.length) {
    case 3:
      r = parseIntHex(hex.slice(0, 1).repeat(2));
      g = parseIntHex(hex.slice(1, 2).repeat(2));
      b = parseIntHex(hex.slice(2, 3).repeat(2));
      break;
      // @ts-ignore
    case 6:
      r = parseIntHex(hex.slice(0, 2));
      g = parseIntHex(hex.slice(2, 4));
      b = parseIntHex(hex.slice(4, 6));
      // falls through
    case 8:
      a = parseIntHex(hex.slice(6, 8));
  }

  return (
    (((a & 0x0ff) << 24) | ((r & 0x0ff) << 16) | ((g & 0x0ff) << 8) | (b & 0x0ff)) >>>
    0);
};

function parseIntHex(value: string) {
  // tslint:disable-next-line:ban
  return parseInt(value, 16);
}

I think you are confusing rgba with argb. The function name already states, that alpha is in front (just another way to represent a color value) which is different from what one would use in css where alpha is usually in the back. Sure - it would be convenient to have rgbaFromHex as well...

@DomiR even so, there is no such thing as a Hex color which has the alpha out front. Hex colors are either RGB RGBA RRGGBB or RRGGBBAA.
I would otherwise agree with you if it weren't for the expectation of a # at the start of the hex string, implying it is not a hexadecimal number, but instead expects a hexadecimal color.


Avoid hierarchal (tree) structure in the textlayer of pdfjs

I have been using Pdfjs for quite a long but with the recent update started to face issues with the textlayer structure. Previously the textlayer was flattened but with the latest release textlayer started to have hierarchy (tree) structure.

Previous version Example:(2.8)

Hello World !!!
I'm confused

Latest version Example:(2.14)

Hello World !!!

I'm confused

any way to get a flat structure on the textlayer?

When opening an issue, please provide all of the information requested in https://github.com/mozilla/pdf.js/blob/master/.github/ISSUE_TEMPLATE.md since as-is this issue unfortunately isn't really possible to easily understand/reproduce.
Of particular importance is that you provide the PDF document in question.

It's possible that you'll need to use the includeMarkedContent parameter, however it is off by default, see https://github.com/mozilla/pdf.js/blob/8bad06f1580f8235c0ab4038d9211587dd39a9b5/src/display/api.js#L1084-L1085 and https://github.com/mozilla/pdf.js/blob/8bad06f1580f8235c0ab4038d9211587dd39a9b5/src/display/api.js#L1643-L1651

sample pdf: https://github.com/BalajiArun004/publicFolder/blob/main/table.pdf
I'm using the pdf-dist CDN URL in a html file to render the pdf file
Expecting a text layer to be flat like this

Hello World !!!
I'm confused

Actual behavior is having a nested span which I want to avoid

Hello World !!!

I'm confused
[table.pdf](https://github.com/mozilla/pdf.js/files/9268270/table.pdf)

to achieve this for now I'm downloading the pdfViewer.js file and changing the includemarkedcontent to be false
link<EMAIL_ADDRESS>is there any way to achieve this without changing in the pdf viewer file

Please attach a pdf (the link above is wrong).

Please attach a pdf (the link above is wrong).

https://github.com/mozilla/pdf.js/files/9268270/table.pdf

<EMAIL_ADDRESS>
That version is no longer supported, please find the latest releases at https://mozilla.github.io/pdf.js/getting_started/#download

is there any way to achieve this without changing in the pdf viewer file

Unfortunately not, since various other parts of the default viewer uses that to improve accessibility.


[BEAM-3981] [WIP] Futurize and fix python 2 compatibility for coders subpackage

This pull request is the result of applying the automatic conversion provided by the future package to the coders subpackage, after which all python 2 errors were fixed. The result is python 3 styled code with python 2 compatibility.
This pull request is the first of a series in which all subpackages will be updated. We should therefore discuss the chosen approach, so we can agree on one strategy to apply throughout.
The approach I've taken, is to focus on writing python 3 code with python 2 compatibility:

The future package provides tools to forward-port our code, while the six package focuses more on backporting. I've therefore replaced six with future everywhere it was already used.

One of the biggest problems in porting python 2 code to python 3, is the changed handling of strings and bytes. To get a consistent behavior between versions, I have tried to rewrite everything to use the str and bytes type provided by the future.builtins package. I have not used the from __future__ import unicode_literals import since its changes are too implicit and introduces a risk of subtle regressions on python 2

I started out with running futurize on the complete coders subpackage and then tried to fix the errors introduced by the automatic conversion. This however proved to be difficult, because it's not obvious where certain errors were introduced.
I therefore switched to a per module approach, in which I first updated all non-test modules. This way, I could check if everything still ran with the native python 2 tests. Afterwards, I updated all test modules.
This pull request also contains updates to run_pylint.sh and tox.ini, so pylint can be run with the --py3k parameter. This should help avoid regression between the different steps of the update process.

cc: @tvalentyn @charlesccychen

Thanks for the review @charlesccychen.
Some general points based on your feedback and my answers:

The imports:
from __future__ import absolute_import
from __future__ import division
from __future__ import print_function
were added at the top of each updated module to prevent regression before full python 3 support is added. This way no new code can be added using for instance the old python 2 division. Another benefit is the consistency of division and print across modules.

from builtins import ... imports from future.builtins on python 2 and has no effect on python 3. future.builtins contains a bunch of backported python 3 builtins for compatibility.
I actually think it's best to add from builtins import * to every module. Just like the __future__ imports, this helps avoid regression and adds consistency. This however still breaks some tests. I'll try to fix these and update the pull request later.

The bytes type annotation was removed in the stream cython files because it was breaking due to a mismatch between the cython bytes type and the future bytes type. However, this is not meant to be merged this way, but I wanted to submit the pull request with working code to get some feedback on this. I have tried replacing bytes with a memory view as explained here, but this resulted in a packaging error. Any help on this is appreciated.

The Cython version was upgraded from 0.26.1 to 0.28.1 because of an incompatibility between cython and future types. I have not noticed any backward incompatibility.

The is type checks were replaced by isinstance checks because the future.builtins are all subclasses of the standard python classes. However, works a lot slower. I could revert this change if I use six again for compatibility. The drawback is that str and bytes will work different across modules.

FAILURE
--none--

I've added some changes. Most notable:

Replace bytes with memoryview in stream cython files. This also works with subtypes of bytes like the future.builtins bytes type.

Revert isinstance checks to typechecks for performance with the use of past.builtins.

I've also added the applied approach to the Python 3 proposal document. It would be great to get some feedback on this so we can start moving forward with the other subpackages.
@aaltay @charlesccychen

I ran into some problems while applying the same strategy to some more subpackages (mostly the typehints package). Most of these problems are caused by the use of future.builtins str, bytes and int.
I've therefore decided to change the used approach, which can be found in the new pull request #5053. A new pull request was made because a lot of the comments in this thread are now irrelevant.
The new approach is unfortunately a bit less resistant to regression, but a better fit for the beam project and I suspect we will be able to move forward more quickly.
Closing this PR in favor of #5053


jira validation tool

check for closed jira references in a pytest repo
check for jira associated with not relevant versions

Report bugs in Issues
The following are automatically added:

Add reviewers from OWNER file (in the root of the repository) under reviewers section.
Set PR size label.
New issue is created for the PR. (Closed when PR is merged/closed)
Run pre-commit if .pre-commit-config.yaml exists in the repo.

Available user actions:

To mark PR as WIP comment /wip to the PR, To remove it from the PR comment /wip cancel to the PR.
To block merging of PR comment /hold, To un-block merging of PR comment /hold cancel.
To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
verified label removed on each new commit push.
To cherry pick a merged PR comment /cherry-pick <target branch to cherry-pick to> in the PR.

Multiple target branches can be cherry-picked, separated by spaces. (/cherry-pick branch1 branch2)
Cherry-pick will be started when PR is merged

To build and push container image command /build-and-push-container in the PR (tag will be the PR number).
To add a label by comment use /<label name>, to remove, use /<label name> cancel
To assign reviewers based on OWNERS file use /assign-reviewers
To check if PR can be merged use /check-can-merge

Supported /retest check runs

/retest tox: Retest tox
/retest python-module-install: Retest python-module-install

Supported labels

hold
verified
wip
lgtm

/wip

@dbasunag
please rename the file get_jira_information.py

/retest python-module-install

@myakove https://github.com/RedHatQE/python-utility-scripts/pull/62#discussion_r1784014584 was already addressed.

/verified


[stable/telegraf] update description

PR in the context of normalizing/improving metadata in the official charts. Refs #124 and helm/monocular#93
NOTE: I have not bumped the chartVersion attribute in order to avoid race conditions/conflicts with existing PRs. Maintainers should be able to change this value before the merge. Llet me know if you prefer that I update it instead.

cc @jackzampolin

Test failure is expected because daemonset.config.outputs.influxdb.urls or single.config.outputs.influxdb.urls is undefined. Feedback is provided by NOTES.txt, which fulfills the requirements of #558. However, I think the feedback could & should be more explicit.
Filed #622 and applying labels.

/lgtm


Convention switch JHUGen

circle Ci fixes and option for convention switch in JHULexicon

@jdavis36 It looks like all checks passed. Are we ready to review?
Question: What happened to the plots_ggH check?

@usarica I still need to add another reference file. The next commit should address that, then we should review.
In regards to the plots_ggH check, these checks take order 5 hours which is way to long for CircleCi. Also, these plots do not contain any information that isn't already contained in the lhe files


Equivalent of chrome://inspect/#workers

Accessing chrome://inspect/#workers gives 404. Does it have an equivalent in Beaker?
Thanks in advance!

We don't currently have that, sorry. Leaving open as a a feature request.

No problem. Thanks!
Btw, FYI, my problem that required this feature is now solved and I'm not in a hurry for this feature :-)


Add text-align: left to button element

As user-agent styles set it to center.

Addressed in "Version 3" updates (see the v3 branch).


Placeholders are not vertically centered in text inputs

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Navigate to staging.new.expensify.com
Check the placeholders in text inputs

Expected Result:
Placeholder should be vertically centered in text input
Actual Result:
Placeholders are not vertically centered in text inputs
Workaround:
N/A, visual issue.
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.0.85-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
View all open jobs on Upwork

From @shawnborton https://expensify.slack.com/archives/C01GTK53T8Q/p1629099710369000

It looks like the placeholder label is not exactly vertically centered in the new text inputs

Triggered auto assignment to @joelbettner (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Proposal
Adding a value of 16 here for expensiTextInputLabel can fix this issue.
(Proposal after Triage)

Proposal
As I mentioned before in Slack
We have to remove top style from expensiTextInputLabel.
Result: centered label

We have to remove top style from expensiTextInputLabel.

Hi, I think removing the property is not a good idea.
I tried that and some UI issues appeared.

This looks like it has what it needs. Un-assigning myself and adding the External label.

Triggered auto assignment to @marklouisdeshaun (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Created a job in Upwork!
Internal: https://www.upwork.com/ab/applicants/1428832340082450432/job-details
External: https://www.upwork.com/jobs/~014ccda5d494c2b148

Triggered auto assignment to @chiragsalian (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

UpWork Proposal
Using vertical-align can solve this problem. You center the box where is the placeholder. Or you can play with the box height to center the placeholder. line-height can be helpful too.

Proposed Solution:
Basically when there is no focus it shows ExpensiTextInputLabel.js component. We need to update top value for expensiTextInputLabel style within src/styles/styles.js file. i.e. change top: 14 to top: 16 as under:
expensiTextInputLabel: {
...
    top: 16,
...
},

So now we moved inactive label default position 2 pt down, So to keep the active label (i.e. when focused) at the same position we have to set ACTIVE_LABEL_TRANSLATE_Y = -12 (at present it is -10) within src/components/ExpensiTextInput/BaseExpensiTextInput.js  (line 12)
I updated both value as I suggested. Below are some screenshot for iOS, Web, and Mac Desktop. At present not tested on android etc. I will test it once solution accepted for further work.
Web

Web - After Login UI
It will also correct input label position everywhere text input used. There is no side effect. Below is the screenshot where text input used after login.

iOS

macOS Desktop

If my solution considered then kindly invite me on Upwork to submit proposal. I will do  final testing, and submit pull request within 4 days once proposal accepted on Upwork.
My Upwork profile: https://www.upwork.com/freelancers/~01c718889ed7821f82
Best Regards,
Prashant Mangukiya

Nice, Yeah i like your proposal @PrashantMangukiya since its the most descriptive, you've tested it well and it looks good to me.
@marklouisdeshaun can you hire @PrashantMangukiya in upwork.
@PrashantMangukiya feel free to create the PR.

Hired in Upwork!

Hi @marklouisdeshaun. Thanks for the job. I accepted it from Upwork. I am doing final testing and preparing for pull request. I will message via Upwork once PR submitted. Also @chiragsalian thanks for the comment.

Hi @chiragsalian  I submit pull request #4823 Kindly review it. If more action need from my side then kindly suggest me. This is my first pull request to expensify so please guide me if I missed something to submit.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to production by @roryabraham in version: 1.0.92-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

Hi @isagoico, This issue deployed to production and passed 7 days period. So I will request to release payment on Upwork for this issue. I already submitted milestone on Upwork. I dont know to whom contact for this so writing message here. Thanks.

Hi @PrashantMangukiya, sorry for the inconvenience! @dylanexpensify is the manager of this issue. In the future, if you have issues with payment, sending an email to<EMAIL_ADDRESS>is a good way to get it quickly resolved. 🙂

Working on this now!

Payment sent!

contract ended and posting removed

@dylanexpensify Thank you for payment.

Of course! Sorry about the delay @PrashantMangukiya !!


[BEAM-12334] Graph java 8 and 11 metrics separately

Fix dashboard query
Graph java 8 and 11 metrics as separately

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @amaliujia

LGTM


[WIP] ACNA-317 - integrate with core error handling sdk

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] 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 change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Codecov Report

Merging #10 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #10   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           5      5
  Lines         272    293   +21
  Branches       57     61    +4
=====================================
+ Hits          272    293   +21

Impacted Files
Coverage Δ

lib/Files.js
100% <100%> (ø)
:arrow_up:

lib/impl/AzureBlobFiles.js
100% <100%> (ø)
:arrow_up:

lib/FilesError.js
100% <100%> (ø)
:arrow_up:

lib/init.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


artifacts-helper alias does not work in shell scripts

Hi, I am using the artifacts-helper to get auth token for .npmrc. It works fine if I run yarn directly from the terminal. However, I see an error when trying to call yarn in a shell script.
error An unexpected error occurred: "Failed to replace env in config: ${ARTIFACTS_ACCESSTOKEN}"
I believe this is because an alias for yarn set in bashrc: alias yarn=/usr/local/bin/run-yarn.sh does not work if I call yarn from a shell script. Currently, as a workaround, I need to have a check if /usr/local/bin/run-yarn.sh exists then run it instead of yarn, something like:
# this is to support codespace (should find a better way to not have codespace specific code in the repo)
if [ -f "/usr/local/bin/run-yarn.sh" ]; then
    /usr/local/bin/run-yarn.sh lint-staged
else
    yarn lint-staged
fi

But I wonder if there is any recommendation to get over this issue to not have this check in all scripts, and also not have codespace specific code in the source repo.
Thanks.

Open to ideas but it is just a hack using bash aliases. Maybe your shell scripts are not using bash? If they are using sh that could be why. Add the she-bang for bash to your scripts would at least not be codespaces specific.

Unfortunately using shebang (e.g. #!/usr/bin/env bash) does not work. I don't think alias will work across either bash/shell scripts, but I'm not sure. Agree that using bash aliases is hacky. I don't have a better idea than calling run-yarn.sh directly in scripts. Will report back if I find a better way. Thanks.

https://github.com/microsoft/codespace-features/pull/65 might help here - in v2.0.2+ functions are used instead of aliases which should behave better.
Another trick that I found useful when running into a similar issue was to use BASH_ENV to set up default shell imports even for non-interactive shells.


[TRA1][T16-B3]

@mervynyeezl Good job for the work. Note that you are failing the codacy test, and try to see which commits failed the tests. Please close the PR after reading this comment.


Content column in document table gets truncated with long length data

When I try to store long length data and I check the document table i see the content gets truncated.
Using sql server as documental bd i expected to store long length data like in a documental db.
the max seems to be 65538 characters
am I doing something wrong?
how could I accomplish this?
Thank you in advance.

Hey @psp589 hello!!
Happy to see you writting here, but sad to read the problem cause it means we are facing another problem in our project.
I'm going to reproduce the problem and debug yessql source to see where exactly is the problem, because at db is shouldn't be because nvarchar(MAX) supports much more characters

the problem is reading the column value with sqlserver but the value is not really truncated


🦟[Bug Report] Crashlytics: DisplayListCanvas.java line 229

Describe the bug

https://console.firebase.google.com/u/0/project/litewallet-beta/crashlytics/app/android:com.loafwallet/issues/263c619f866800c901def3ec590ca25e?time=1672490238000:1675082238000&sessionEventKey=63D79CA6001B00016E5D250E77AA847A_1772836774900554112
Additional Info

This is likely due to an image that is no longer used

Defunct Image 1
Defunct Image 2

thoughts @vsima ?


Add app version to the About dialog - fix issue #106

One step closer, but I'm not sure about having to update the version in all 4 pom.xml files...
Another issue is with a version like 1.4.0 without the -SNAPSHOT sufix, the app does not build.

@stefansbv Concerning the version in all pom.xml - we can define a single "version" property in the parent pom.xml and just use that property in all our pom.xmls

I'll take a look at problems with versions without -SNAPSHOT
It should be 1.5-SNAPSHOT (1.4 was released)

Hi @sebbrudzinski, false alarm, it builds fine using 1.4.0 or 1.5 as version, sorry.

@sebbrudzinski, I don't know how you make the releases, but it looks like the versions in POMs can be easily incremented with the maven-release-plugin from the CLI.

@stefansbv I'm aware of the maven-release-plugin. Thanks for notifying though! :) Is it possible to have the version in parent pom properties anyways?

@sebbrudzinski it looks like it is required to have the version in parent pom properties. When the version is incremented, all 4 poms (pom.xml, Editor/pom.xml, application/pom.xml and branding/pom.xml) must be updated with the same version.

There are conflicts on this branch that need to be resolved.

Forgot to cleanup imports...
@sebbrudzinski  does travis-ci have some issues? I see "Some checks haven’t completed yet" from my previous commits.

It won't build as long as there are unresolved conflicts

I think it was a problem with Travis-CI at the time I pushed the commits and the job have never finished, it has nothing to do with potential merge conflicts.

Or maybe it had... :)

@stefansbv It seems like something is not quite right about this pull request. I can see a bunch of unrelated commits and changes. This needs to be fixed.

Yes, I see...
I am thinking to do a hard reset to Simplify version read as suggested by @sebbrudzinski and take it from there. What do you thing?

Made a cleaner pull request.


Add "leading sponsor" setup to sponsors display, fixes #44

The Issue

#44

Platform.sh and future "lead sponsors" want a more prominent listing.
Review at https://20241108-add-leading-sponsor.ddev-com-front-end.pages.dev/#supporters
Manual Testing Instructions
Review the main page in regular and dark mode.

dark:border-white
For dark mode we probably want the border white.

OK, trying it out, thanks!


Added fictitious delete methods, changes for android build

Fictitious delete methods solve the problem of calling a nonexistent method in finalize

We are going to finish the guide and once we understand that code is valid for Android device we will merge it.


Cannot preview the theme locally

HI,
I configured my _config.yml and Gemfile as directed, but when I run
bundle exec jekyll serve
I got these error?
Configuration file: /home/kyao4/kyao4.github.io/_config.yml
Configuration file: /home/kyao4/kyao4.github.io/_config.yml
            Source: /home/kyao4/kyao4.github.io
       Destination: /home/kyao4/kyao4.github.io/_site
 Incremental build: disabled. Enable with --incremental
      Generating...
     Build Warning: Layout 'post' requested in _posts/2017-05-20-welcome-to-jekyll.markdown does not exist.
  Liquid Exception: Could not locate the included file 'icon-github.html' in any of ["/home/kyao4/kyao4.github.io/_includes"]. Ensure it exists in one of those directories and, if it is a symlink, does not point outside your site source. in about.md
jekyll 3.4.3 | Error:  Could not locate the included file 'icon-github.html' in any of ["/home/kyao4/kyao4.github.io/_includes"]. Ensure it exists in one of those directories and, if it is a symlink, does not point outside your site source.

Can someone explain to me why this is happening?

@kyao4, this error does not seem to be related to the slate template.
It looks like you have a line in your 'about.md' file similar to:
{% include icon-github.html %}

so jekyll is looking for an 'icon-github.html' file in your "/_includes" folder, which doesn't exist. If you find the above line of code and take it out, you should stop seeing that error.
For more on how the include tag works check out the jekyll docs.
The slate template does not have any reference to this file, so it's probably from something else you're using or trying to do.
Hope that helps.

@asbhat Oh, Your answer is indeed helpful.
I used command
jekyll new myblog
to create my blog.
and then edited _config.yml and Gemfile accordingly.
I think what I should've done is to clone this repo and customize it by myself?

@kyao4 ah, I see what you did now. Yes I can recreate this error.
Are you on macOS?
If so, do
$ open $(bundle show minima)

This will open a finder showing you all the files within the minima gem (the default theme). Navigate to the "_includes" folder you see.
You can copy all of those files to an "_includes" folder in your "myblog" folder.
There's more information on handling theme changes here: https://jekyllrb.com/docs/themes/
Taking a step back, I would suggest you first go through this guide that jekyll recommends: http://jmcglone.com/guides/github-pages/


Adding iterative merge sort

ooooooooooooof💀
This must be like Sorting->Python->your_file.
Try looking at contribution guidelines before making PR

Okay will keep that in mind next time.

Thanks for your contribution. Append your name to the contributors.md file following the mentioned format.


AssertionError: folder "\path\to\audio\files" does not exist

Hi ,I am trying to implement musiclm by following the instructions.
However, I got an error "AssertionError: folder "\path\to\audio\files" does not exist" when I was trying to train the three transformers.

I wonder where I can download those files ?

wherever your files are?
its a place holder?


Add ability to get/set signals untracked

Closes #85

get is implemented, just need to do set

Thanks! I would love some tests... I need to go to bed but if you want to add some you can model them on the ones here and use the untracked setters/untracked getters to make sure effects aren't rerunning.
Otherwise I can add tests in the morning and merge if it all looks good.

@gbj I was going to try and impl UntrackedGettableSignal for Signal and MaybeSignal, but I don't think this is possible. I wonder if there might be another way to go about doing this? Such as perhaps adding some kind of property to the scope that says "Hey, we are going into an untracked state, any subscriptions that come in, ignore them", which may work universally for any signal kind?

I agree that it's not possible on Signal and MaybeSignal because there's no way to do an untracked get on a derived signal (as opposed to doing untracked gets of the signals within that derived signal, if you see what I mean)
The more general solution already exists with Scope::untrack(), which can run an arbitrary closure without tracking anything.
Having separate untracked get/set on signals makes sense, because
a) it's more ergonomic to write a.get_untracked() than to write cx.untrack(move || a.get())
b) it's faster to use get_untracked as you implemented it than to go through the work for Scope::untrack

I never saw the Scope::untrack() method, but we can use this meanwhile which should work for Signal and MaybeSignal?

For example, something like this:
fn get_untracked(&self) -> T
    where
        T: Clone,
    {
        match &self.0 {
            SignalTypes::ReadSignal(s) => s.get_untracked(),
            SignalTypes::Memo(m) => m.get_untracked(),
            SignalTypes::DerivedSignal(f) => {
                let f = f.to_owned();
                let val = Rc::new(RefCell::new(None));

                let val_clone = val.clone();

                let disposer = create_scope(move |cx| {
                    *val_clone.borrow_mut() = Some(cx.untrack(|| f()));
                });

                disposer.dispose();

                let mut val_borrow = val.borrow_mut();

                val_borrow.take().unwrap()
            }
        }
    }

Would this actually work? I would hope yes, but not sure if the Scope::untracked would actually not track the get since it belongs to a parent scope.

I haven't tried it but I'd be very surprised if that worked; create_scope there is creating an entirely new reactive system, so I don't think you'll actually be able to access the values that are within the derived signal. It would probably panic as you're trying to access signal IDs that don't exist.
The best way to do this would actually be to change Signal::derive so that it's fn derive<T>(cx: Scope, derived_signal: impl Fn() -> T) -> Signal<T> and change Signal so that it has a variant DerivedSignal((Scope, Rc<dyn Fn() -> T>).
Then you could just do
fn get_untracked(&self) -> T
    where
        T: Clone,
    {
        match &self.0 {
            SignalTypes::ReadSignal(s) => s.get_untracked(),
            SignalTypes::Memo(m) => m.get_untracked(),
            SignalTypes::DerivedSignal((cx, f)) => cx.untrack(f)
        }
    }

I just checked and the docs for create_scope are completely wrong; I must have mixed them up with the child_scope docs at some point and not noticed.

@gbj we should be good to go, although I haven't written any tests yet

Thanks, I'm fixing the doctests and adding some tests, and will push and then merge if CI passes.

All set. Thanks so much for the PR!


Exclude carrel patron barcodes from patron lookup

Addresses a problem for requesting items via Borrow Direct when the patron has an active carrel barcode.

@kevinreiss @tampakis This should be good to go. I added a description of the patron groups to be excluded.


Warning: Undefined array key "class" in src/Drupal/ServiceMap.php on line 41

I get this when running against core, e.g: https://dispatcher.drupalci.org/job/drupal_patches/91829/console
I think the culprit is autowire_test.services.yml:
services:
  Drupal\autowire_test\TestInjection:
    public: false
  Drupal\autowire_test\TestInjectionInterface:
    alias: 'Drupal\autowire_test\TestInjection'
    public: false

If a service does not have a class key then the class is the service name itself, we do not use this pattern (yet) in Drupal but Symfony allows it.

Ah! Yep, that's a bug. I didn't account for auto wiring since Drupal didn't support it. Thanks for the catch!

This is fixed in 1.0.1 :)


pip install -r requirements.txt on MacOS

I installed Python 3.7.2 and update alias for python and pip to point 3.7.2 version
Could not find a version that satisfies the requirement mlagents_envs<0.7,>=0.6.2 (from obstacle-tower-env==1.2->-r requirements.txt (line 1)) (from versions: )
No matching distribution found for mlagents_envs<0.7,>=0.6.2 (from obstacle-tower-env==1.2->-r requirements.txt (line 1))
System
MacOS 10.14.3
Python 3.7.2 (Python 3.7.2 (v3.7.2:9a3ffc0492, Dec 24 2018, 02:44:43)  [Clang 6.0 (clang-600.0.57)] on darwin)
pip 19.0.2 (pip 19.0.2 from /Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/pip (python 3.7))

Hi @iamchathu
This error is because mlagents_envs currently does not support Python greater than 3.7.0. I will look into enabling python 3.7.x support for the next release.

@iamchathu Just to be clear, the current workflow requires Python 3.6.x, so you can either downgrade your OS python, or preferably set up a conda environment with Python 3.6 for the Unity Challenge.

hello,
any updates on this issue?
same problem here too.... why this old python version?
when will you update it for last python ?
Collecting aicrowd-repo2docker (from -r requirements.txt (line 2))   Could not find a version that satisfies the requirement aicrowd-repo2docker (from -r requirements.txt (line 2)) (from versions: ) No matching distribution found for aicrowd-repo2docker (from -r requirements.txt (line 2))
sry i don't get it why there is a old version for to use this...

Hi @MadMacMad
You can try with Python 3.7 by downloading the mlagents_envs package from here: https://pypi.org/project/mlagents-envs/, and changing the python requirement in setup.py. Please let us know if this works for you.

Hi @MadMacMad
You can try with Python 3.7 by downloading the mlagents_envs package from here: https://pypi.org/project/mlagents-envs/, and changing the python requirement in setup.py, and installing it directly. Please let us know if this works for you.

doesn't work.... give up on this ... no time to figure it out... time is money... and if this doesn't work i don't want to waste my time with lame issues because some don't care about OS X @ Unity3d...
it's a old problem since adding the PC platform in Unity 3.6... I'm Unity user since beginning... now i have a feeling everything went down because of making money... look into the forums... only ghost are there

Hi @MadMacMad
I am sorry that you have had a bad experience. On our team at Unity we have tested the workflow on Mac, Windows, and Linux platforms. As I mentioned, the issue is only with Python 3.7. Python 3.5 and 3.6 can be downloaded and used on these platforms. It is also possible to run a python virtualenv in order to isolate your python installation, if you are worried about package incompatibilities.

You can install it with the following command:
pip install --ignore-requires-python https://files.pythonhosted.org/packages/e5/e1/b6d26f1c799c5c8d4c3968008faa12a46ab9f404831c50441ff57739b4a2/mlagents_envs-0.6.2-py3-none-any.whl


Make the button classes overridable

Hey there,
So I noticed that you really can't add any extra classes if you need to do so, I think that would come in handy time to time

That variable is internal, you should not set classes through that variable but through drupalAttributes if you want extra ones.

Thanks for the update tibi


Add go based operator and update bpfd k8s APIs

This PR sets up the scaffolding for writing the bpfd-operator (and maybe bpfd-agent) in golang.
Additionally it migrates and updates the bpfd CRD definitions.  Firstly it moves the definitions to be in golang in order to take advantage of the existing k8s go based k8s api tooling.
It also adds two new Objects

EbpfProgram Object (To keep track of per node bpfd metadata for each loaded program)
EbpfProgram Meta (To keep global state regarding an ebpf program which has been loaded to all or some nodes in a kubernetes cluster)

A first stab at an architecture diagram can be seen below

TODO:

[ ] Remove existing API CRDs
[ ] Convert existing rust based bpfd-agent to work with the CRD updates
[ ] Remove any extra autogenerated fluff code we don't need from the bpfd-operator directory

@Billy99 I'm sure there's some typos in this as I was doing this with sleepy eyes :) but I wanted to go ahead and get this posted since I'm in meetings almost all morning tomorrow.
PTAL whenever you get the chance

/lgtm
This looks good as a first draft so we can start working on the go based operator without stepping on each-other @Billy99


unable to work google format plugin at windows

Hello
Is there a way to work cosium google format plugin at windows machines ? It works properly in UNIX systems but it does not work at Windows Machines ? Do you know the reason behind that ?
Thank you

Duplicate of #59


test PR with Github API

Please pull this in!

Docs Build status updates of commit 54debeb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

tutorial-manage-data-disk.md
:white_check_mark:Succeeded

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


Question

@Low-Drag-MC How did You fix an issue with Chocolate Quest Repoured region protection to not cause crash with Metallurgy 4 Reforged while loading a world ?

https://github.com/Davoleo/Metallurgy-4-Reforged/issues/371
I thought that this is Metallurgy 4 Reforged issue, but this mod's developer wrote that this is CQR issue in their code.


I can search the duty free page for a resource

Description:
As a Concourse user
I can see a search bar at the top of the duty Free page
And I can type in a search term and search for a resource on the catalog page
So that I can find the resource or resources that match my query
Acceptance Criteria
Given I am on the Duty Free Page
I can see a search box at the top of the page
When I type a term into the search Box (e.g. Slack)
I see the list of resources that match the search term (e.g. Slack Reading and Posting)
PM Notes: We have agreed to do a very basic search that will allow us to do user testing

Accepted by @Pivotal-Kevin-Curley. Behaviour is as per the acceptance criteria.


[gce_export] Export raw: support exporting to local disk

A new param "local_path" is added. Now it's a either-or between "local_path" and "gcs_path".
The reason to add this is because we plan to change the way of exporting raw. This PR is the 1st half of the change. The 2nd half would be: change export workflow to use "local_path" and then do "gsutil cp" to gcs. Between 1st half and 2nd half, we need to release a new debian-9-worker image to adopt the new gce_export.
The purpose is to avoid long TCP connection with GCS. Currently, when directly writing .tar.gz file to GCS, a super long TCP connection to write the whole file (by io.CopyN) has to be established. However, it's not as stable as before (tcp connection reset happens a lot), and it's anyways unstable while writing a large image.
Per my testing, the performance of new implementation is the same as the current one.
The cost is higher due to more disk consumed, but no more than $0.1 for a 200GB exporting.

Moved log out from sub func

Part of README that describes how no local file is created is outdated now

You are right, I just modified accordingly.


Expand support to include FAT12

What kind of effort would it take to expand support to include the FAT12 file system?

Hey,
just have a look here: https://github.com/waldheinz/fat32-lib/tree/master/src/main/java/de/waldheinz/fs
That library has FAT12 and FAT16 support. I think it would also possible to use that library instead of mine pretty easy.
At the moment I do not have time to add FAT12 support sorry.
But I am curious, why do you need that? I thought that only FAT32 would be needed, so I just left FAT12 and FAT16 out :D

I needed it because the device I was integrating with was FAT12. My requirements were such that I only needed read-only capabilities for one specific file on the device, so it only took a few days. The biggest differences were the lack of partition tables and of course, the 12 bit FAT entries.
Thank you for the library. It was very helpful!

@mjdev Hi,is there any library for android otg for support a FAT12? I wery need it. Or can you show example for use https://github.com/waldheinz/fat32-lib/tree/master/src/main/java/de/waldheinz/fs in android project?