[Feature Request]: Allow sideinput for batch DoFn

What would you like to happen?
When using .side_inputs(...) in a ParDo with a DoFn using process_batch, we get :
DoFn <__main__....> has unsupported process_batch method parameter SideInputParam
Why can't we get access to the sideinput from inside the process_batch function ?
in sdks/python/apache_beam/runners/common.py
in _validate_process_batch
  # Fallback to catch anything not explicitly supported
      if not d in (core.DoFn.WindowParam,
                   core.DoFn.TimestampParam,
                   core.DoFn.PaneInfoParam):
        raise ValueError(
            f"DoFn {self.do_fn!r} has unsupported process_batch "
            f"method parameter {d}")

Issue Priority
Priority: 2
Issue Component
Component: sdk-py-core

over to @TheNeuralBit to triage.

Thanks for feedback @alex4adeo, it's good to know what users are trying to do. Let's track this under #21653 (SideInputs are a per-key param).


MultiOr's \bottom and MultiAnd's \top should not lose information

When the inner type is Pattern, for example, the sort information gets lost.

I think the best way to handle this is to change the behavior so that when a MultiAnd or MultiOr is \top or \bottom respectively, we keep the \top or \bottom element; but only if it is the singular element of the (con,dis)-junction.
I also thought about adding a Sort field, but then the types wouldn't be usable with things that don't have a canonical sort, and I don't like that.

After #1442, most of the uses of the phony _PREDICATE sort will go away anyway, so we decided to wait until the relevant pull request is merged and deal with the remaining uses by removing mkTop_ and mkBottom_. We will have to keep track of some sorts directly in the code (rather than tracking them in the MultiAnd data structure), but we will need to keep track of the sort there anyway.


Add support to partition keys in supported components when publishing

EventHubs and Kafka (maybe others) have the option to define a partition key when publishing an event. By defining a partition key producer can ensure event ordering for related items (i.e. events from user '123' end up in same partition).
Proposal:

Using optional "partitionKey" in output binding/publisher metadata to define a value. This would handle scenarios where the key is a string. Scenarios where the partition key is another type wouldn't be supported.

Would be glad to jump into it, if you feel it is a good addition.

Hi. There is a broader discussion on redesigning the PubSub API for dapr. This is one of the new features. Please, include this in the discussion and close this as duplicate. I see that you are part of that discussion too.
Reference: https://github.com/dapr/dapr/issues/843

Moving discussion to dapr/dapr#843

BTW, Maybe you can tackle this as part of item number 3 in dapr/dapr#843. It will create a way for the publisher to add optional special fields that Dapr will understand. One of them is PartitionKey, another one is MessageID and more to come.


Why use separate API endpoint for listProvisioningProfile? [Question]

New Issue Checklist

[√] Updated fastlane to the latest version
[√] I have read the Contribution Guidelines

Issue Description
As stated in the Spaceship Readme file, the Provisioning Profile entities are handled using 2 separate API endpoint:

https://developerservices2.apple.com:
-- Get a list of all available provisioning profiles
...
https://developer.apple.com:
-- Create new certificates, provisioning profiles and apps

Looking at the code, this introduces a big deal of modifications to the provisioning_profile class, compared to the rest of the classes, especially for handling CSRF tokens.
I was playing with Spaceship and made it use the same API endpiont (developer.apple.com) for listProvisioningProfile.action, and couldn't spot any difference in the outcome.
I also could not find a comment in the code or earlier commits referring this difference.
Is there a specific reason for using https://developerservices2.apple.com there?

@EliyahuStern I'm going to close out this issue. We've recently switched over to using the developer.apple.com api, however one major difference is that theappId isn't included in the listProvisioningProfiles call so we're having to make an additional request to getProvisioningProfile for each profile right now to get that data.


align nnet decoder & refactor

PR types

[New features]
PR changes

Describe

align nnet decoder & refactor

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


Fix to combine async_dup Arc and Mutex to avoid thread issue in test

Closes #370
Description
Fixes the field request and response of TestCase to type Arc<Mutex<Unblock<File>>>.
And clones them directly in function clone.
Is it a feature that change user experience?
No
Client output
No

Pull Request Test Coverage Report for Build 317593164

8 of 9 (88.89%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 89.484%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/protocol/src/tests/async_io.rs
8
9
88.89%

Totals

Change from base Build 314093795:
0.2%

Covered Lines:
8186

Relevant Lines:
9148

💛  - Coveralls


Creature Type not Generating Proper Tag

Under 'Type' the correct creature type is listed, but it adds an 's' to the end of the type and does not correlate with an Alchemy creature Type tag. This also applies to the size and the alignment, but those may need to be separate tickets.

This was a combination of a few small issues. The way I was reporting monsters and monster types was slightly different than alchemy was expecting. After examining the 5e srd version of monsters I fixed the gaps and updated internal monster type maps to better align with alchemy's expected types. This will be published in 0.2.1 final


Revert "Added Article on Choosing the Correct Channel Type"

Reverts MicrosoftDocs/windows-uwp#119

@mijacobs,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


object detection with vertex automl

Solution notebook for training and deploying object detection model with Vertex AI AutoML.

LGTM

LGTM

Great lab! LGTM.


Data Paper

Description
@sepandhaghighi @kasraaskari Please check out the data paper and comment.
Dataset of standard tests of Nafion 112 membrane and Membrane Electrode Assembly.pdf

Thanks for your effort 💯
My comments :

We should mention somewhere in data paper that this dataset published under CC BY 4.0 license
Reported Data title should move to next page
We should avoid from chartjunk, I think Figure(9-2), Figure(10-2) and Figure(4-2) should be removed or reformatted
Figure(6-2) has wrong caption position
All tables header should have unit
Repository of PEMFC reported dataset contribute to ECSIM/pemdataset1 development by creating an account on GitHub, this sentence is vague!!

@sepandhaghighi Thanks for your comments. All mentioned cases except third case, was edited. I need more time to work on charts issue.

@sarminh Thanks for your effort ;-)


Avoid CDATA in CDATA

If something writes out CDATA into the log, this logger crashes with the following error message:

MSBUILD : error MSB4017: The build stopped unexpectedly because of an unexpected logger failure.
Microsoft.Build.Exceptions.InternalLoggerException: The build stopped unexpectedly because of an unexpected logger failure. ---> Microsoft.Build.Exceptions.InternalLoggerException: The build stopped u
nexpectedly because of an unexpected logger failure. ---> System.ArgumentException: Cannot have ']]>' inside an XML CDATA block.
at System.Xml.XmlTextWriter.WriteCData(String text)
at MSBuild.ExtensionPack.Loggers.XmlFileLogger.WriteMessage(String message, Boolean escape)

To avoid this I have simply removed CDATA boundaries from the message.
You can test it by using this MSBuild task:
<Message Text="<![CDATA[ something]]>" />

Sorry for the tabs...


optimize optional chaining compile

Suggestion

🔍 Search Terms

List of keywords you searched for before creating this issue. Write them down here so that others can find this suggestion more easily and help provide feedback.
✅ Viability Checklist

My suggestion meets these guidelines:

[ ] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[ ] This wouldn't change the runtime behavior of existing JavaScript code
[ ] This could be implemented without emitting different JS based on the types of the expressions
[ ] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[ ] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion

evaluate the type before optional chaining, if it's a string type for example, just generate code like (a.b || '')
avoid generate lots of ternary expression that make codebase bloated
📃 Motivating Example

💻 Use Cases
 const a= {
     b:''
 }
// 1. specific api call like String.prototype api
// maybe the most useful use case
 console.log(a.b?.slice())   --->  (a.b || '').slice

// 2. get value（maybe a breaking change since undefined --> '' empty string）

{a?.b}
   --->
{(a.b || '')}

console.log(a.b?.slice())   --->  (a.b || '').slice

These have different results. a.b?.slice() results in undefined if a.b is nullish, and (a.b || '').slice() results in an empty string.

fair enough, maybe the developer should constraint it by self


Masked echo functionality

I’d like a way to provide per-keystroke feedback, such as printing "•" or "***" for each character that the user enters. (I have some users who are just getting accustomed to terminals, and this is one of their first interactions.
Even better, something like this:
pub fn read_password_with_hook<F>(hook: F) -> std::io::Result<()>
where F: FnMut(char) -> std::io::Result<bool>;  // Ok(true) means continue

would let me write my thing (quick and dirty):
fn read_with_echo(echo: &str) -> Result<String> {
    let mut result = String::new();

    read_password_with_hook(|c|
        if c == '\n' {
            tty_print_raw!("\n")?;
            Ok(false)
        } else {
            if c != '\r' {
                result.push(c);
                tty_print_raw!("{}", echo)?;
            }
            Ok(true)
        })?;

    Ok(result)
}

If you like this idea, I’m happy to implement it and submit a PR. There’s probably a better design than read_password_with_hook, and I’m happy to discuss that too. Are you interested?

Given this issue is now 6 months old without change, I'll go ahead and close it. Feel free to reopen it you want to keep working on this.


Resend a invitation email when expired does NOT update created/sent_at

When an invitation link sent by mail is expired, re-deliver the invitation doesn't update the created_at and sent_at timestamp and so all the new tokens are not valid.

By invitation I imagine you're using devise-invitable? There's nothing about invitations in devise itself.
Please clarify, and if it's a devise issue add some code to show what might be going on, so that we can try to help. Meanwhile I'll close this, and I'll gladly reopen if it's devise related. Thanks.


optimize alert query; revert incorrect millisecond to second conversion fix

During unsuccessful attempts to get our alert queries running on the app-sre grafana dashboard, I did come up with a simpler query which reaches the same basic effect of only using the last 30 minutes of data available.  This change will also work better with the use of range seen in other successful app-sre dashboard queries.
Also, I realized an earlier change by me  to divide our milliscecond metric's _count by 1000 was incorrect.  Only the _sum has actual millisecond based observations that need to be converted (as was originally done).
The PR where I incorrectly changed what @ramessesii2 had for converting milliseconds to seconds was https://github.com/openshift-pipelines/pipeline-service/pull/835
Here is a screen shot of how the new queries look in RHTAP red-hat prod:

task deploy-pipeline-service-baseline has the status "Failed":

Normal  ResourceUpdated   9m46s  argocd-application-controller  Updated health status:  -> Missing
      Normal  ResourceUpdated   85s    argocd-application-controller  Updated health status: Missing -> Degraded
command terminated with exit code 1

/retest test-pipeline-service-upgrade-ocp-412


RESTful games/create

Checklist

[x] Use a separate branch in your local repo (not master).
[x] Test coverage is 100% (or you have a story for why it's ok).

Coverage remained the same at 100.0% when pulling 62e7ec481ebf8818536483c78b9a683ba408f982 on amitport:RESTful into 2959acd38d3f23cf7965704a56cbbc14eeae6b1a on google:master.


Add a role to configure masquerading on hypervisor

Using nat type networks in libvirt causes firewall rules to be inserted by libvirt - these rules block traffic between libvirt networks. In some scenarios this is not desired, we want traffic between internal networks and still enable NAT (masquerading) of the traffic going to the external (internet/intranet).
This role's task is to insert these firewall rules to masquerade traffic where the src is a libvirt network and the dstis external (internet/intranet).
Also enable it in the reproducer. For example:
cifmw_masquerade_networks: true
cifmw_masquerade_external_source_ranges:
  - "{{ cifmw_networking_definition.networks.ironic.network }}"
  - "{{ cifmw_networking_definition.networks.ctlplane.network }}"

As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role
[x] Content of the docs/source is reflecting the changes

recheck

/approve
though I really don't like having two ways to manage the firewall :(.

Moving this to draft - after reading https://discussion.fedoraproject.org/t/f41-change-proposal-libvirt-virtual-network-nftables-self-contained/120329,  I think there may be a way to do this with firewalld.

Closing this, superseeded by https://github.com/openstack-k8s-operators/ci-framework/pull/1928 which is using firewalld.


MOTECH-2660: Loading state for motech-list

Fixed searching by "search" field

Can one of the admins verify this patch?

test this please

test this please


WIP: v4

In this major version, we would drop the support for node 4 (#408) in order to use the built-in Promise instead of bluebird (#639 ).
Other plans are:

[ ] Use TypeScript to rewrite this module for cleaner code and official type decroations
[ ] Upgrade redis-parser to v3
[ ] Support Per-Request retry limitations.
[ ] Easier way to iterate the key space via SCAN and ES6 Iterators.

Cluster related:

[x] Refresh slots automatically #641
[ ] Support for buffer keys #637

Any other ideas? @shaharmor @Salakar @AVVS

Cluster:

Refresh slots upon a disconnection from one of the cluster nodes
Restructure the cluster initialization code so that the initial slots refresh call will be done only after connecting to the first node. This will help with issues like #581 where the user is setting enableOfflineQueue: false which causes the slots refresh call to fail immediately and disconnect from the node.
Consider setting enableOfflineQueue: true explicitly on the internal cluster nodes - @luin correct me if I'm wrong, but the offline queue of the internal node shouldn't be used in cluster mode because when ever the socket will disconnect the node will automatically be removed.

Other:

Remove callback support? Not sure why we have to support callbacks if we support promises. If anyone really needs callback support, they can "callbackify" the promises themselves. Why do we need the entire library to support it?

Refresh slots upon a disconnection from one of the cluster nodes

A disconnection won't lead the update of slots immediatially. A cluster will consider a master to be failing only after NODE_TIMEOUT has elapsed.
So most of time we only need refetch the slots in two different situations (https://redis.io/topics/cluster-spec#clients-first-connection-and-handling-of-redirections):

At startup in order to populate the initial slots configuration.
When a MOVED redirection is received.

However, given a failing of a slave won't trigger a MOVED error, so refresh slots intervally help us solve the problem.

Remove callback support? Not sure why we have to support callbacks if we support promises. If anyone really needs callback support, they can "callbackify" the promises themselves. Why do we need the entire library to support it?

Good idea. I've seriously considered removing callback support. However, what makes me tend to keep callback support in this major version is that many built-in functions (fs.readFile) still follow callback-style, and util.promisify was absent until v8.0.0. The major frameworks (e.g. express, mocha) continue to be friendly to the callback style, so there should be many legacy codes prevent developers from migrating to the ioredis v4 if we drop callback support at this moment.
We may do this on the next major version, which only supports node >= v8 that support async functions.

Just release a beta version v4.0.0-0. The remaining issues will be resolved in the next beta version.


Stuck, help plz

I am seemingly stuck.. need suncrest, don't have lighter either for two items in candle room... something I'm missing? Was no item on bed in serum saferoom. Thanks!
Download Visual Studio Code to view DGML file so that I may break down item code location but not having much luck in deciphering this.
log_pl1.txt
Seed: R510-VHCK-J6ABCKX01ZZ1

The sun crest is in room 212, which is Forrest's grenade launcher. Did you check this, was it the grenade launcher or something else? Did the Barry cutscene play, and did he give you a different item?

Thanks for quick response. Forest gave me the grenade launcher and Barry gave me acid rounds.

I wasn't able to reproduce this. Forrest gave me the sun crest.

ok, going to run a few more tests.. starting to think I may need fresh install, this isn't my first time being soft locked, even going back to a randomizer that pre-dates yours.. Barry always gives me acid and Forest always has launcher, like in vanilla, wood crest is always wood crest.. I assume that's normal though. Thanks again

Mixing randomizers is also not a good idea. So yeah make sure you have a vanilla install.
The wood crest is not randomized as doing so prevents the gold one from being placed.


Cascade the DISABLED state if all dependencies are either DISABLED/DONE ...

...and at least one is DISABLED.
This PR will set a task to be DISABLED if any of its dependencies are DISABLED, and the rest are either DISABLED or DONE.
This allows workers to terminate when something is DISABLED, even if worker keep-alives are enabled. WIthout this patch, keep-alive workers never terminate even if tasks are DISABLED, effectively causing the workflow to hang.
The approach taken here means that as much work as possible is done, and only the critical path is disabled (i.e. we "fail slow" instead of immediately bailing out if anything gets DISABLED).
This sort of relates to #331 but not exactly, but does address the note at the end of #684.

Tests are usually nice :)

I'll do the tests if it gets the thumbs up for a merge. Don't want to do unnecessary work if this is somehow against the direction the maintainers want to go in :-)

I'll do the tests if it gets the thumbs up for a merge. Don't want to do unnecessary work if this is somehow against the direction the maintainers want to go in :-)

Ok, makes sense @boosh !
@erikbern ? I can't say I know enough about the scheduler to sya if this is a good idea.

OK lemme take a look

@erikbern Where did we get to with this? If I write tests, are you happy to merge in?

I forgot this. Do you still want to merge it? Let me take a look

This still doesn't handle re-enables, so it'll take a bit of re-writing.
On Sun, Mar 8, 2015 at 12:29 PM, Erik Bernhardsson<EMAIL_ADDRESS>
wrote:

I forgot this. Do you still want to merge it? Let me take a look
—
Reply to this email directly or view it on GitHub
https://github.com/spotify/luigi/pull/686#issuecomment-77769249.

I can work around this for now by running a cron job that checks that the final output of a job was created by a certain time, so if other people think this may complicate how luigi currently works, we should leave it for now.

It should be fairly simple to get the effect you want. Just keep track of
upstream status and don't count upstream disabled jobs toward keeping
workers running. The only issue I have is that marking them as disabled
rather than upstream disabled will have other negative effects.
On Tue, Mar 10, 2015 at 5:24 AM, boosh<EMAIL_ADDRESS>wrote:

I can work around this for now by running a cron job that checks that the
final output of a job was created by a certain time, so if other people
think this may complicate how luigi currently works, we should leave it for
now.
—
Reply to this email directly or view it on GitHub
https://github.com/spotify/luigi/pull/686#issuecomment-78043116.

should we close this then?

I'll close this because I don't have the time to look at it right now. If I need this in future I'll make the changes suggested by @daveFNbuck and reopen the PR.


Configure order for routes added by platform-http consumer

This conversation lead to an interesting discovery about how the Vert.x router route order can affect the behaviour of the platform-http consumer.
TL;DR the user set up a REST route which expected to match method=POST & Content-Type=application/json. When the correct content type was not provided by the client, a 404 response was returned instead of  the expected 415.
Turns out that because META-INF/resources was present in the source tree, Quarkus registers Vert.x route(s) to handle the static resources. These are registered after the Camel ones. Thus the 415 response is silently dropped because the static route handlers can't match anything and return a 404.

Do we need to do something about this?

I did start working on a solution. But after this conversation I abandoned it.
If it becomes a frequent issue for folks, then maybe we invest more time on it.


Second instance FIPS 140-2

Second instance per @kimsauce  that FIPS 140-2 compliance is not available for Cloud Syslog in the FedRAMP deployment.
Purpose of this pull request
This pull request (PR) ...
Issue number:

Select the type of change:

[x] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

@stacykor , did you mean to edit the AWS GovCloud file here? The Cloud Syslog edit was handled in your previous PR.

@stacykor, did you mean to edit the AWS GovCloud file in this PR?
The Cloud Syslog edit you did here was handled in your https://github.com/SumoLogic/sumologic-documentation/pull/2172, so this PR is rendering as blank because the change was already done.

@stacykor, did you mean to edit the AWS GovCloud file in this PR?
The Cloud Syslog edit you did here was handled in your #2172, so this PR is rendering as blank because the change was already done.

I'll look in the morning. I was trying to grab the second instance.

@stacykor I will close this one out since it's not needed


Adds CAS Common API tutorial, fixes #5.

Needs a reviewer maybe before merging with main. Thanks @stuchalk !

Vin, thanks for this I will take a look.

Generally, there needs to be text to explain what the code is doing in each block, where there is none currently.  This does not need to be much. Something allowing the user to understand what each code statement translates to would be really helpful.
Specifically,

A little more in the introduction indicating that the different sections get more and more complicated because of the amount of data retrieved...
In block the fourth block of section 2 'vew' should be 'view' (comment)
In Section 3 'Handle' should be 'Handling' (in two subtitles)
Could you add a couple of suggestions for other things to try? Nothing big, but asking to apply what they have learned...

Additional checking of the Python code reveals that 'import json' present in three places is not used and thus can be deleted. In addition, the 'import numpy' in the last section is also not used...and so should be removed.

Vin, if you can update the PR by removing the updated _toc.yml file, then I can work on it.  We are not expected contributors to edit the TOC file ;)

Thanks, TOC change should be removed now.


When is the next release?

Hi, There were fixes included we need for the next release. May I know when it is going to be released? I don't see a release happened this tuesday, i.e. Aug 14th, as you guys generally release every Tuesday.

@ashygk You are right. We did not release this week. We are going to make a release on Monday-Tuesday next week. Do you need this release urgently?
Thank you,
Andrew

@andrewtelnov Thanks for getting back. Yes, we were waiting for the release.

@ashygk How bad are you waiting it? I am the only person in the office for today and tomorrow, vacation time :)
Could it wait till Monday, or do you need it badly?
Thank you,
Andrew

Our last sprint failed because of the updates we were waiting for. We are in the new sprint now and still waiting to close them after receiving the update. Could wait till Monday, but would appreciate if available at the earliest. Thanks :)

@ashygk Got it.
Thank you,
Andrew

@ashygk v1.0.38 is available.
Thank you,
Andrew

@andrewtelnov Thank you so much!

@andrewtelnov  Could you please re-open issue #1259 in surveyjs issues . The issue has been closed without verifying and the fix doesn't work


Merge pull request #513 from OfficeDev/master

Merge to live!

:white_check_mark: Validation status: passed
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.


ci: rework multi-arch build

Some fixes to the currently failing Docker build workflow:

rather than relying on caching which wasn't set up properly, build and push specific arch images with their own cache, then merge them into usable tags in the merge job
setup proper GitHub token permissions for the whole workflow
add latest tag to the latest Docker builds on main

There are some example runs on my fork: https://github.com/SamyDjemai/tailscale-manager/pull/2
We can see that, using caching, the ARM64 build job drops from 3 hours to 4 minutes! (We will avoid these long build times when ARM64 hosted runners will be available for OSS projects).

Builds are failing due to insufficient GITHUB_TOKEN permissions as explained in this documentation. Should we set the workflow as approval only? Or should we restrict Docker image building/publishing to only happen on the main branch?


update blog & cli folder docs for MDX3

Description of your changes

I have:

[ ] Read and followed KubeVela's contribution process.
[ ] Update sidebar.js if adding a new page.
[ ] Run yarn start to ensure the changes has taken effect.

Special notes for your reviewer

thanks


Confirmation toasts

users need feedback when they do an action. Toasts will serve nicely.

Here is the documentation for the style I'll use:
https://getmdl.io/components/#snackbar-section


Bug: Outdated Twitter Logo on Aiida home page

Issue:
The Aiida home page currently displays an outdated version of the Twitter logo. The logo being used is not in sync with the latest branding guidelines.
https://github.com/aiidateam/aiida-core/assets/120818337/d4ac49dd-3f87-4ae9-88c2-f5f444530011
Steps to reproduce
Visit the Aiida website on chrome .
Navigate to the section displaying social media links in the navigation menu.
Observe the Twitter logo and compare it with the latest Twitter branding guidelines.
Expected Result:
The Twitter logo should adhere to the most recent branding guidelines provided by Twitter.
Actual Result:
The current Twitter logo appears to be outdated and may not align with the latest branding standards.

Fixed by #56


HsColour on GHC.Prim fails

I just encountered this while building master,
/--------
| Running GhcCabalHsColour with arguments:
|   hscolour
|   libraries/ghc-prim
|   dist-install
\--------
Running hscolour for ghc-prim-<IP_ADDRESS>...
Preprocessing library ghc-prim-<IP_ADDRESS>...
ghc-cabal: can't find source for GHC/Prim in ., dist-install/build/autogen
Error when running Shake build system:
* libraries/ghc-prim/dist-install/doc/html/ghc-prim/ghc-prim.haddock
user error (Development.Shake.cmd, system command failed
Command: /opt/exp/ghc/ghc/inplace/bin/ghc-cabal hscolour libraries/ghc-prim dist-install
Exit code: 1
Stderr:
ghc-cabal: can't find source for GHC/Prim in ., dist-install/build/autogen
)

I tried to reproduce this on my Windows machine but ghc-cabal hscolour libraries/ghc-prim dist-install was successful, as well as all other invocations of GhcCabalHsColour builder.
I've pushed a small related fix to deal with absolute paths on Windows: 30d3d63. Currently configure sets the following path in system.config: hscolour = /c/Users/[...] which I replaced with C:/Users/[...] to be able to run hscolour. Maybe there is a better way to deal with this.

I can say that Prim.hs certainly does not exist where I would have expected it,
$ shake-build/build.sh
...
/--------
| Running GhcCabalHsColour with arguments:
|   hscolour
|   libraries/ghc-prim
|   dist-install
\--------
Running hscolour for ghc-prim-<IP_ADDRESS>...
Preprocessing library ghc-prim-<IP_ADDRESS>...
ghc-cabal: can't find source for GHC/Prim in ., dist-install/build/autogen
Error when running Shake build system:
* libraries/ghc-prim/dist-install/doc/html/ghc-prim/ghc-prim.haddock
user error (Development.Shake.cmd, system command failed
Command: /opt/exp/ghc/ghc/inplace/bin/ghc-cabal hscolour libraries/ghc-prim dist-install
Exit code: 1
Stderr:
ghc-cabal: can't find source for GHC/Prim in ., dist-install/build/autogen
)

$ ls libraries/ghc-prim/dist-install/build/autogen/
cabal_macros.h  Paths_ghc_prim.hs

Ah, that's true. I can now reproduce this. We need to add a dependency on autogen/GHC/Prim.hs (this file is generated by a rule in Rules/Generate.hs).
If you amend you pull request #8 I'll fix this using your partial solution (otherwise we'll have merge conflicts).

Not sure why this got closed automatically when merging #8...

What is the status of this?
My impression was that 8e8cc53 is the correct solution here. HsColour depends upon the source. Settings.getPackagesSources should already handle the addition of autogen/GHC/Prim.hs.

Sorry, full rebuild takes a huge time on my machine (I need to find out why).
I think you are right and 8e8cc53 should be enough but I'd like to double check before closing this.

OK, I've tested this and I think we can close this now.
A minor relevant fix: 0c9d7d8.


Headful and Headless mode have different tab indexes

When running Puppeteer on Headful mode, whenever I click something that opens a new tab I can use const newPage = await browser.targets()[3].page().
However, this doesn't work in headless mode, as const postPage = await browser.targets()[3].page() is actually my first page, and const postPage = await browser.targets()[4].page() becomes the new page.
Why am I getting this difference between headless/headful mode?
Steps to reproduce
Tell us about your environment:

Puppeteer version: 1.0.0
Platform / OS version: Ubuntu 16.04
URLs (if applicable):
Node.js version: 8.9.4

What steps will reproduce the problem?
Click something that opens a new tab;
Its index will be different in headless and headful mode.
Please include code that reproduces the issue.

Open new tab
Try to access that tab with the same index for headless and headful mode.

What is the expected result?
Being able to work with the same code for both modes.
What happens instead?
Can't find correct page.

// Clicks a button that opens a new tab
await page.click('#someButton')
await page.waitFor(1000)

// When running HEADFUL MODE, target is = 3
// When running HEADLESS MODE, target is = 4
const newPage = await browser.targets()[4].page()
await newPage.type('textarea[aria-label="Edit title"]', title)

@gtaltemann there are many different targets other then page, e.g. extensions and workers. Extensions don't exist in headless mode but do exist in headful. So in your case, browser.pages() should work.
In any case, I'd refrain from using indexes to address pages in browser and instead find the right one based on the url.
Hope this helps.


Persian language Support for Gemini API

Description of the feature request:
gemini  support persian language in public use but does not support it in API  this cause limitation for making worldwide app i hope you add persian support to gemini API
What problem are you trying to solve with this feature?
make worldwide apps

Yes, good request

Yes, please consider this request

Great request. please consider.

Yes. Please consider.

Please add persian language api

Yes. please consider I need this.

@Sinazrp, Thank you reporting this issue.
This repository is for issues related to website(https://ai.google.dev/) like documentation bugs or improvements. For issues related to Gemini API, we would suggest you to use "Send Feedback" option in Gemini docs. Ref: Screenshot below.

Yes, please consider this request

Its now working

Its now working

It's not API,

Its now working

It's not API,

It is, i tested with api


suppressing property kind warning for length-thickness mismatches

This change suppresses the property kind warning when the kind and expected kind are both one of 'length' or 'thickness'.

Codecov Report

Merging #220 (e241073) into master (863eea3) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #220      +/-   ##
==========================================
- Coverage   49.02%   48.97%   -0.05%
==========================================
  Files          53       53
  Lines       25284    25261      -23
==========================================
- Hits        12395    12372      -23
  Misses      12889    12889

Impacted Files
Coverage Δ

resqpy/property.py
48.60% <100.00%> (-0.03%)
:arrow_down:

resqpy/olio/base.py
84.54% <0.00%> (-0.80%)
:arrow_down:

resqpy/crs.py
67.90% <0.00%> (-0.27%)
:arrow_down:

resqpy/lines.py
57.31% <0.00%> (-0.17%)
:arrow_down:

resqpy/time_series.py
61.80% <0.00%> (-0.10%)
:arrow_down:

resqpy/weights_and_measures.py
95.23% <0.00%> (-0.08%)
:arrow_down:

resqpy/organize.py
58.88% <0.00%> (-0.05%)
:arrow_down:

resqpy/well.py
47.62% <0.00%> (-0.05%)
:arrow_down:

resqpy/grid.py
39.66% <0.00%> (-0.05%)
:arrow_down:

resqpy/surface.py
56.57% <0.00%> (-0.04%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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


License

From what I saw from this site https://dev.maxmind.com/geoip/geoip2/geolite2/, the db files are under the Creative Commons Attribution-ShareAlike 4.0 International License, but this repository is under MIT.
Can I use this as a MIT dependency even if the CC BY-SA 4.0 states:

ShareAlike — If you remix, transform, or build upon the material, you must distribute your contributions under the same license as the original.

?

@runk

This is a very good point. Will fix it!


Fixes Ion 1.1 symbols for opcodes E1-E3

Issue #, if available:
None
Description of changes:
Prior to this change, Op codes E1 - E3 were being handled incorrectly because of several problems:

Ion 1.0 and Ion 1.1 symbol values with SIDs were both being treated as if they were encoded as a (big endian, Ion 1.0) UInts.
IonReaderContinuableCoreBinary.readFixedUInt_1_1() would start reading at peekIndex and stop at valueMarker.endIndex, but peekindex was not necessarily being set.
The IonTypeID for Opcode E3 has a length of -1, and when reading the next value, IonCursorBinary was blindly setting valueMarker.endIndex to one less than the start index.

Changes:

symbolValueId() now branches based on the Ion version. If it is Ion 1.1, it reads FixedUInt rather than UInt.
IonReaderContinuableCoreBinary.readFixedUInt_1_1() now accepts a start and end index.
If an Ion 1.1 IonTypeID doesn't have a length prefix (i.e. variableLength == false), and it has length == -1, (and it's not delimited), then slowReadValueHeader() and calculateEndIndex_1_1() now assume it to be a FlexInt or FlexUInt, and the length is calculated by looking at the continuation bits of the value.
Adds Ion 1.1 SID symbol value test cases to IonReaderContinuableCoreBinaryTest.

Why didn't we notice this sooner? Lack of proper unit testing, and maybe a bit of (bad) luck. Most of our test suite has symbol tables with less than 256 symbols, which means that (a) for the SIDs we were testing, there was no practical difference between big and little endian since they were only one byte long, and (b) we were not exercising opcodes E2 and E3 in any meaningful way.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Why didn't we notice this sooner?

We just hadn't implemented the feature yet, then we forgot about it. That explains the lack of tests. Everything was just following the existing 1.0 code path.

Spotless always compares against changes from master?

Yes. It does. If I can figure it out, I will change it to use the commit from master that is the closest ancestor of the current commit—that way divergence between master and a feature branch won't cause sudden failures like this.


override public ip prefix name in aks global

Have verified that if I don't add the variable this change don't perform any changes to an existing environment.


paths with - (dashes) are not correctly read

when I have a command that takes a path if I use a path containing a dash surrounded by escapes it is interpreted as an option
e.g.
@nef_app.app.command()
def stream(
file_name: Path = typer.Argument(..., help='input file')
):
app ${HOME}/OneDrive\ -\ University\ of\ Kent/file.txt
Error: No such option: -
inputs: stream /Users/garythompson/OneDrive\ -\ University\ of\ Kent/file.txt
exiting...
adding quotes doesn't improve things...
app ${HOME}/'OneDrive - University of Kent/file.txt'
Error: Got unexpected extra arguments (- University of Kent/file.txt)
inputs: stream ${HOME}/OneDrive - University of Kent/file.txt
exiting...
click should be able to correctly deal with filenames that contain dashes and escaped spaces
for example cat is quite happy to stream the same file and grep can grep the contents
Environment:
Python version:   3.10.0 (default, Oct 27 2021, 09:04:48) [Clang 12.0.0 (clang-1<IP_ADDRESS>)]
Operating system: macOS-10.15.7-x86_64-i386-64bit
Click version: 8.1.3

This is your shell, not something click acn control. You need to properly quote tokens containing spaces and dashes.

The following example works for me:
import click

@click.command
@click.argument("name", type=click.Path())
def main(name):
    click.echo(name)

main()

$ python example.py 'a - b'
a - b

oops I think this is typer rather than click

tried your code on my machine  and it seems fine, I presume its something to do with typer


remove terratest

This PR removes unused Terratest tests in test/ directory in lieu of existing Terraform Tests in tests/ directory. Test Framework

/do-e2e-tests


Messed up

I am using the latest version of jspdf and all the characters are messed up with strange spacing and characters appearing.
I have seen so many people raising the same issue with screenshots and examples that look just like mine. Is there a solution to this?
Thank you!

I guess you are using Unicode
#12


Custom filters (MSFilter) doesn't work with Swift 3.0

I'm having the same issue.  Log the issue with Microsoft a week ago.  No updates so far.

@dmgerdes Thanks for reporting this to us. We are working on it.

The issue with MSFilter should be resolved by upgrading to XCode 8.1. As long as you're using XCode 8.1 or above, you should be fine.
Btw, we released v3.3.0 today and it now officially support Swift 3.0.


source command not working via exec method.

Node version (or tell us if you're using electron or some other framework):
7.7.4
ShellJS version (the most recent version/Github branch you see the bug on):
0.7.7
Operating system:
Ubuntu 16.04
Description of the bug:
Trying to execute the source command using shell.exec() but it isn't working. I get source not found error.
/bin/sh : 1: source: not found

Example ShellJS command to reproduce the error:
var shell = require('shelljs');
shell.exec('source somefile.sh');

source isn't supported from exec.
If you want to execute a script, invoke it directly: shell.exec('bash name-of-script.sh'). This isn't very portable, which makes me wonder if ShellJS would provide any advantages for you.
If you really need the to source a script like somefile.sh (e.g. to modify your environment), you'll need to transpile that to ShellJS as well. You can take a look at https://github.com/nfischer/shelljs-transpiler for help.
If this doesn't answer your question, please ping this issue and we can reopen it.


textfairt build issue

hello sir when i build this project it shows this errors
Build command failed.
 Try moving the project to the root of the C drive or try building this on a mac instead.


Autolayout

Creates the RoomLayout class, which can be retrieved from any room via Room.getLayout(roomname) or Room.prototype.getLayout().

The RoomLayout class stores layouts as costmatrixes in a dedicated segment (using sos.lib.vram).

RoomLayout can be manipulated with planStructureAt, getStructureAt, getAllStructures, clear, save, isPlanned. The save function has to be called when the layout is changed or those changes will be lost.

RoomLayout.visualize will display the planned structures.

The program CityLayout has templates for layouts, currently a core and flower template. It attempts to place them on the map- It tries to keep the structures groups close together and close to the controller, but relaxes those requirements if they don't work.

There is some level of randomness in the layout- running the same layout code multiple times will result in different layouts. This lets us "brute force" the layout by attempting it multiple times, rather than seeking the "optimal" solution.

Autobuild is not part of this pull request

This Pull Request has been merged by GitConsensus.
Vote Totals

Yes
No
Abstain
Voters

3
0
0
3

Vote Breakdown

User
Yes
No
Abstain

tedivm
✔

new-guy
✔

PostCrafter
✔

Vote Results

Criteria
Result

Has Quorum
True

Has Votes
True


nixos/neovim: fix withRuby, add with{Python3,NodeJs}

Motivation for this change
withRuby was declared but not referenced before, this commit fixes this issue.
It also adds support for withPython3 and withNodeJs.
Fix issue https://github.com/NixOS/nixpkgs/issues/144830.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

CC @teto.

it's a bit infuriating to have this code duplication between home-manager and nixos. Let's hope some knight in shining armor will tackle the issue someday.


Usage of numpy.random.Generator to deal with Data-Loader Parallelism

Hi @duncanriach
Thanks for an awesome repo, which helped to fix a number of issues in my code.
For now I can can achieve reproducible results running on GPU with TF 2.4.1 and single worker. Which is good, but painfully slow.
You mentioned, that that there is an option to define pseudorandom number generator state for each parallel thread running. Do you have any code example of how to do that using TF 2.0+?
It seems that spawning of workers is handled internally and only way to interact with the process is define num_parallel_calls value in dataset.
Best regards,
Jamil

Hi Jamil (@zakajd),
Thank you for the appreciation. What method of tf.data.Dataset are you calling with the num_parallel_calls parameter?.
Duncan

Currently train and val datasets use the following methods:
train_dataset = (
    train_dataset
    .cache()
    .map(read_tfrecord, num_parallel_calls=AUTOTUNE)
    .shuffle(512)
    .map(
        lambda o, e: (resize_and_rescale(o), resize_and_rescale(e)),
        num_parallel_calls=AUTOTUNE
    )
    .batch(cfg.training.batch_size , drop_remainder=True)
    .map(
        lambda o, e: augment(o, e, training=True), num_parallel_calls=AUTOTUNE
    )
    .prefetch(AUTOTUNE)
)
val_dataset = (
    val_dataset
    .cache()
    .map(read_tfrecord, num_parallel_calls=AUTOTUNE)
    .map(
        lambda o, e: (resize_and_rescale(o), resize_and_rescale(e)),
        num_parallel_calls=AUTOTUNE
    )
    .batch(cfg.training.batch_size, drop_remainder=False)
    .map(lambda o, e: augment(o, e, training=False), num_parallel_calls=AUTOTUNE)
    .prefetch(AUTOTUNE)
)

When setting AUTOTUNE parameter to 1 results are reproducible between different runs on GPU and CPU (but results between GPU and CPU are slightly different). When setting AUTOTUNE=4 or AUTOTUNE=tf.data.AUTOTUNE each run results in a different model.

Hi @zakajd, I suspect that my original guidance on this may have come from data augmentation pipelines that did not use tf.data.Dataset. One was to solve this problem with tf.data.Dataset is to serialize the generation of pseudorandom parameters and then pass those into the computationally-expensive data augmentation process, which can then be arbitrarily parallelized. Here is some example code:
import tensorflow as tf
import numpy as np

np.set_printoptions(precision=2, floatmode='fixed')

def augment_random(x):
  random = np.float32(np.random.uniform())
  # The addition here represents a computationally-expensive set of operations
  return np.float32(x) + random

def random_param():
  return np.float32(np.random.uniform())

def augment(x, y):
  # The addition here represents a computationally-expensive set of operations
  return tf.add(tf.cast(x, tf.float32), y)

def nondeterministic_pipeline():
  np.random.seed(123)
  dataset = tf.data.Dataset.range(1, 6)
  dataset = dataset.map(
      # From tf.data.Dataset::map documentation: Note that use of
      # tf.numpy_function or tf.py_function in general precludes the possibility
      # of executing user-defined transformations in parallel (because of the
      # Python GIL).
      lambda x: tf.numpy_function(augment_random, inp=[x], Tout=tf.float32),
      num_parallel_calls=5)
  return np.array(list(dataset.as_numpy_iterator()))

def deterministic_pipeline():
  np.random.seed(123)
  dataset = tf.data.Dataset.range(1, 6)
  dataset = dataset.map(
      # From tf.data.Dataset::map documentation: Note that use of
      # tf.numpy_function or tf.py_function in general precludes the possibility
      # of executing user-defined transformations in parallel (because of the
      # Python GIL).
      lambda x: (x, tf.numpy_function(random_param, inp=[], Tout=tf.float32)),
      num_parallel_calls=1)
  dataset = dataset.map(lambda x, y: augment(x, y), num_parallel_calls=5)
  return np.array(list(dataset.as_numpy_iterator()))

result1 = nondeterministic_pipeline()
result2 = nondeterministic_pipeline()
result3 = deterministic_pipeline()
result4 = deterministic_pipeline()

print("\nGenerate random parameters in parallel:")
print("run 1: ",result1)
print("run 2: ",result2)

print("\nGenerate random parameters in series:")
print("run 1: ",result3)
print("run 2: ",result4)

# Generate random parameters in parallel:
# run 1:  [1.55 2.72 3.29 4.70 5.23]
# run 2:  [1.72 2.23 3.55 4.29 5.70]

# Generate random parameters in series:
# run 1:  [1.70 2.29 3.23 4.55 5.72]
# run 2:  [1.70 2.29 3.23 4.55 5.72]

I have also updated the documentation for deterministic data-loader parallelism to cover this topic.

I just discovered that the solution that I suggested above has been suggested before, on github/tensorflow/tensorflow issue 13932.
Also, now closing this current issue.

Thanks!
I ended up doing something similar, but used tf.random.uniform and implemented a stateless random operations similar to tf.image.stateless_random_contrast.

The relatively new stateless random image ops, such as tf.image.stateless_sample_distorted_bounding_box can also be used with this approach: a seed-per-example is generated in a single-worker stage and used with these ops in later, parallelized stages.


returnOld & returnNew query options do not seem to be supported on vertex collection HTTP requests

Same problem as #2363, except now on vertex collections.

When this issue is fixed, remove all lines like this from the go-driver tests. (test/vert*.go)
skipBelowVersion(c, "3.3", t) // See https://github.com/arangodb/arangodb/issues/2365


edits based on tips and feedback from daniel

command > Shell

done

Marcus Behrens, Director Product Management, SAP IoT, +49 160 8896 474
I respond to SMS, Teams and WhatsApp in 1 hour, email in 1 day or never
From: Daniel Wroblewski<EMAIL_ADDRESS>Date: Thursday, 8. October 2020 at 16:56
To: SAPDocuments/Tutorials<EMAIL_ADDRESS>Cc: Behrens, Marcus Conrad<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [SAPDocuments/Tutorials] edits based on tips and feedback from daniel (#5950)
Can you change the "command" language for the code blocks to "Shell"?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/SAPDocuments/Tutorials/pull/5950#issuecomment-705626898, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACKKGUFIUN5D6DIGBIGHYHDSJXHI5ANCNFSM4SI3MRNQ.


About testing result

hello, i test the results use the prerain-model you offered, but i find the result is very worse .In some clear lane mark picture also not good,i want to know why.And is this model your last testing model? And 是否存在坐标的转换问题，我获得的测试结果，利用SCNN的tools生成结果图的。下面是我其中一张。

@cardwing Look  the pic is ok ,but in the real world picture is not good.

@Endless-Hao, what is the size of your input image? And do you test the pre-trained model and get the final F1-measure (71.3)?

@cardwing input pic is CULane data size 1640590 ,and get result is from SCNN-tensorflow is 800288. And then i put them ,use the SCNN-original tools. Get the results like the picture i put in the web.

@cardwing  Do you mean i should change the test input size to from 590 1640 to  288 800?

@Endless-Hao, you should first check if the F1-measure of the pre-trained model is 71.3. The original input image should be ok.

@cardwing  hello! I use SCNN original evalution get these:
file: ./output/vgg_SCNN_DULR_w9_iou0.5.txt
tp: 0 fp: 111702 fn: 0
precision: 0
recall: -1
Fmeasure: 0
All the step i follow your step.Also i change the input to 288 800,i get the result still not good.i don't know why.

The folder are like these:
predicts_SCNN_test_final
vgg_SCNN_DULR_w9
home
pch
test
driver_37_30frame
driver_100_30frame
driver_193_90frame
06042010_0511.MP4
00000.exist.txt
00000_1_avg.png
00000_2_avg.png
00000_3_avg.png
00000_4_avg.png

@Endless-Hao, can you attach the original folder here? I need the whole file to check which part is wrong. You can put the link which stores the generated probability maps here.

@cardwing  hello  you are a good 老哥 Thanks.This is my original folder:
problem.zip

@Endless-Hao, I have found that the generated probability maps are different these of mine. You can either test my generated probability maps or re-train the model and have a test then.

@cardwing  i look all the results, also i change the input ,but get different result which are all not good. Do you konw is it roughly what caused it? I use the data and model the same as you. If there is not solution  at last ,i think  i must retrain it. I just feel strange about this. The steps are same as you.

@Endless-Hao, I have downloaded the pre-trained model and regenerate the probability maps. The results are still good and the F1-measure is exactly the same as what is listed in the table in README. I recommend that you should check the evaluation process as well as the probability map generation process.

@cardwing  i have another problem, can you debug. I want to see the every step. Your code in my computer ,can not debug. When i debug, it just say"pyder debugger: process 26490 is connecting" only this sentence.

@cardwing  i also  find, when i test it. If there is no line in the picture, the exist.txt also  1 1 1 1. This is also strange thing.

@cardwing  i use you result you upload. I find that your results is very good. The last generate picture is very good.

the results you upload actually good.

@cardwing  but i can not get these. Do you have any little  change in your code you forget it ?

@Endless-Hao, everything is ok in my local server, including debugging and running (I download the codes and perform testing). I also check the uploaded codes and find nothing is wrong. The main difference between your generated probability maps and mine is the position of detected lanes. You may have to re-train the model and check the performance then.

@cardwing  i also find a strange condition. Every time i test the pictures, the same model and the same code will get the different results.

@Endless-Hao, it is weird. You should check the code carefully since it is impossible to get different outputs using the same input and same model.

@cardwing  this is what  i am so strange. If  this can not solve, i think i will also get not good results to use the model that  i train my own . I continue to check the code. Thinks. If i get the new condition, i would talk with you. Thank you for your help.

^_^

@cardwing i also find, when i test it. If there is no line in the picture, the exist.txt also 1 1 1 1. This is also strange thing.

Hello, I have the same problem as you. How did you solve this problem?

@cardwing  i also find, when i test it. If there is no line in the picture, the exist.txt also 1 1 1 1. This is also strange thing.
@Endless-Hao ,Hello, I have the same problem as you. How did you solve this problem?


OverflowError: cannot fit 'int' into an index-sized integer

Here is the minimal test case:
from theine import Cache

cache: Cache = Cache(policy="tlfu", size=10000)

def test1() -> None:
    print(len(cache))
    cache.clear()

def test2() -> None:
    print(len(cache))
    cache.clear()

I am running this test file using pytest. Second test fails with OverflowError: cannot fit 'int' into an index-sized integer error. If I remove cache.clear() then it works. If I change the policy to "clockpro" then it also works.
Environment: MacOS 13.5.1, python 3.8.16.

@extesy please upgrade theine-core to v0.4.2 and try again 🙏

@Yiling-J That fixed the problem. Thank you for a quick response!


Configurable LSP transport

Currently, the VS Code GraphQL extension enforces IPC as the communication transport between the language client and the server. This is only one of the 4 communication channels recommended by the LSP specification.
Both the client and the server support stdio as a communication channel, so this change makes it configurable. This can be convenient in those environments where IPC communication may not be available.
The default transport is unchanged.
A setting is added to allow users to quickly change the communication channel:

Tested via the VS Code LSP Extension Run functionality in VS Code:

Added packages/graphql-language-service-server/src/__tests__ to the workspace
Changed the setting to stdio
Opened the test.graphql file
Got auto-completion
GraphQL Language Server Output looks correct

good idea! originally vscode only supported IPC, wasn't aware that they support this. or other LSP users outside of vscode use these transports every day so they should indeed be ready to go

@acao Great to hear. Anything missing or can this be merged, then?

@acao
$ yarn run vsce:package
yarn run v1.22.22
warning<EMAIL_ADDRESS>The engine "vscode" appears to be invalid.
$ vsce package --yarn
Executing prepublish script 'yarn run vscode:prepublish'...
warning<EMAIL_ADDRESS>The engine "vscode" appears to be invalid.
$ npm run compile -- --minify

><EMAIL_ADDRESS>compile
> node esbuild --minify

successfully bundled vscode-graphql 🚀
 DONE  Packaged: [REDACTED]/graphql/graphiql/packages/vscode-graphql/vscode-graphql-0.12.1.vsix (127 files, 51.92MB)
 INFO
The latest version of @vscode/vsce is 3.2.1 and you have 2.29.0.
Update it now: npm install -g @vscode/vsce
✨  Done in 11.82s.

Installed the .VSIX and everything seems to be working fine. Just a typo I'm going to fix now.

good eye, I didn't even see the typo! you tested the extension when switching the modes and the server restarted appropriately, yes?
this should be good for now, we should add the socket transport as well but I can test that

I reloaded the window with both settings and verified the server started in both cases and provided auto-completion.

@acao We are good to go from my side, if you are happy.


Spinning up, list of previous efforts

Let's start this by collecting where the previous efforts were actually located.
Here's one: https://github.com/nodejs/io.js/pull/2111
cc @nodejs/smoke-test

I've used https://github.com/misterdjules/test-node-apps to run jobs on jenkins.nodejs.org that would test some popular (according to me) modules/apps' tests. This program does a few things that I found useful at the time, such as transforming various tap outputs to be parsable by Jenkins, allowing us to specify additional npm dependencies, or applying transformations to the package.json file.
Here's a sample result page: http://jenkins.nodejs.org/job/nodejs-v012-apps/98/DESTCPU=ia32,label=ubuntu-12.04/tapTestReport/.
Originally, I wanted to have an idea of what applications were broken by Node.js v0.12.0. After a while, the Jenkins job was using too much resources, and because of lack of time I made the decision to disable the job.

This has probably been stated in other places, but we want modules that are:

Can be tested using nothing but npm install and npm test. It would be ideal if all of the tests were passing so that we're not trying to figure out what broke because of io.js and what is just broken.
Popular / in heavy use
Represent a good variety - frameworks, utilities, JS modules, native modules

Anything else?

we want modules that are:

Can be tested using nothing but npm install and npm test.

FWIW, my experience when I was running the Jenkins job mentioned in my previous comment seems to indicate that while it's an ideal situation, very often the testing workflow of many projects is not that straightforward. It often involves using global dependencies (for instance, Grunt, phantomjs, etc), or tests' output that is not TAP, or not proper TAP. There's also sometimes a mix of unit tests that are easy to run in a sandbox, and integration tests that sometimes need a more complex setup (which can sometimes be automated).

I can echo that. Each of the modules I've been testing have to be scripted slightly differently, most conform to npm i && npm t but then you have to deal with varying output and exit codes (mocha + docker + exit codes = wth???). https://github.com/rvagg/iojs-smoke-tests/tree/master/tests

We (IBM) have had a prototype mechanism written initially in python that's been on my plate to get reimplemented in Node for a while now. I think I need to buckle down over the next week or two and just get that done. Each test does need to be scripted independently, which is quite irritating but we've got to work with what we've got. Going with a docker based approach would be great, but it only gives us coverage on the Linux based distributions. It won't tell us if things aren't working elsewhere for whatever reason. Given that, I'd like to explore an approach that spins out tests on multiple lightweight vms where each target platform can be smoke tested. Reporting the results is tricky because of all the different ways outputs are handled from project to project. Again, we'll have to work with what we've got initially and hope we can nudge developers in better directions later on.

In case that wasn't clear: (1) we want to be able to run the smoke test on each of the primary platforms we support, (2) that we're able to keep each test isolated and scripted and (3) that we're able to normalize the output for each test

Are we only concerned with tests for releases and whatnot? Or might we also be interested in a something that can run as part of the regular CI? Any other potential use cases?
For CI, I wonder if it might be useful to identify a single important package (e.g., request or async) that hopefully has decent tests. If it's a small enough package, the test might reliably run in under a minute. ( I imagine the full smoke test suite on all platforms will take hours to run.) A CI smoke test like that can let us know if we are ever again implementing some seemingly innocuous change that will completely break the npm ecosystem. That's kind of what I was going for with https://github.com/nodejs/io.js/pull/2111. Is that potentially A Good Idea or kinda misguided?

Maybe good reference, here's the smoke-testing impl for Rust: https://github.com/brson/taskcluster-crater#crater---the-rust-crate-tester


Update connection.json to avoid pointing to not yet existent entities

Right now the connection.json in the repo uses a role/warehouse/database/schema that don't actually get created until after the user successfully connects to snowflake in their notebook. Taking out the role/warehouse/database/schema seems to fix this and then in the next few code blocks after connecting, these all get created and explicitly set.

Accepting the suggested improvement.


Update zh translation in pod-security-policy.md

Update zh translation in pod-security-policy.md

Page in sync, good to go.
/lgtm
/approve


Update package.json

A package withou RC Tag looks much better. Especially when there have been no changes for two years.

Thanks, 2.0.0 has been released


Type error in deploy.ps1 - [DBOpsConfig] class cannot be referenced

Importing the dbops module in the deploy.ps1 file does not load the DBOpsConfig class referenced on line 46.
Issue should be solved by sourcing . "$PSScriptRoot\Modules\dbops\internal\classes\DBOps.class.ps1"

Thanks for bringing this up, this is clearly a bug. The class is supposed to be for internal use only and it simply shouldn't even be mentioned in the deploy file.

Resolved in 0.5.5.


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/OPS-E2E-PPE/docs-archive-test

Docs Build status updates of commit c15e8f9:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-09-25-07-57-26 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-09-25-07-57-26

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.


boundary: 0.1.8 -> 0.2.0

Motivation for this change
Bump boundary to 0.2.0
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 119535 run on x86_64-linux 1

1 package built:

boundary


Stopped working on my Macbook Pro

I'm not sure what's going on but when I restarted my Macbook Pro Atom windows wouldn't open.  An error box popped up saying: The editor has crashed Please report this issue to https://github.com/atom/atom. I know it is up to date because I am very diligent about that sort of thing. I even went as far as downloading Atom again from the Atom website, deleted my current version and tried replacing it with the newly downloaded version. I got the same message. Any help would be GREATLY appreciated as I use ATOM daily! (I really like Atom and I don't want to have to switch to another text editor.)

@fman29 Sorry to hear you're having trouble!  Can you try out the following things for me and see if they work?

atom --safe
atom --clear-window-state (note: this will irreversibly clear Atom's window state.  If you have any unsaved changes, those will be wiped out.  If you think that's not worth trying, then feel free not to.)

In addition, what is the output of atom --version?
Finally, in the future, it would be greatly appreciated if you could follow the default issue template that is provided to you when creating an issue.  It helps us provide faster and more directed feedback :).

Unfortunately, I can't get it to open ANY window. I've got the Atom menu bar and nothing else. Every attempt to open any window or search preferences or config files fails.
  From: Wliu<EMAIL_ADDRESS>
To: atom/atom<EMAIL_ADDRESS>Cc: Kevin<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Sent: Friday, November 18, 2016 6:20 AM
Subject: Re: [atom/atom] Stopped working on my Macbook Pro (#13259)
@fman29 Sorry to hear you're having trouble! Can you try out the following things for me and see if they work?

atom --safe
atom --clear-window-state (note: this will irreversibly clear Atom's window state. If you have any unsaved changes, those will be wiped out. If you think that's not worth trying, then feel free not to.)In addition, what is the output of atom --version?Finally, in the future, it would be greatly appreciated if you could follow the default issue template that is provided to you when creating an issue. It helps us provide faster and more directed feedback :).—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@fman29 Please follow the steps laid out by @50Wliu, and answer his questions.
Right now, we do not have enough information to help resolve your problem. Unless the requested information is not provided in a reasonable timeframe, this issue will be closed.

I had the same issue.

atom --safe does work once I've closed all windows. I can then open any project. My react package was not parsing jsx files as it usually does.

$ atom --version
Atom    : 1.12.3
Electron: 1.3.6
Chrome  : 52.0.2743.82
Node    : 6.3.0

Now that I've opened atom in safe mode once, I cannot reproduce the issue, even when opening without safe mode, everything is fine. Issue is solved for me at least!

I have the same problem but atom works only if I use atom --safe with atom themes.
Atom    : 1.12.4
Electron: 1.3.9
Chrome  : 52.0.2743.82
Node    : 6.5.0

Thanks everyone! Your instructions seem to have worked like a charm! I appreciate your very quick responses and all of your help!!

If Atom works in safe mode, try disabling community packages until it works in non-safe mode.

Hi All,
I am unable to open Atom editor on my Mac.
Below are the details
Shailenders-MacBook-Pro:Root shailenderkumar$ atom --version
Atom    : 1.19.5
Electron: 1.6.9
Chrome  : 56.0.2924.87
Node    : 7.4.0
Shailenders-MacBook-Pro:Root shailenderkumar$ atom --clear-window-state
2017-09-02 20:39:57.749 atom[78436:9788664] *** Assertion failure in +[SQRLDirectoryManager currentApplicationManager], /Users/joshaber/Documents/Development/GitHub/Squirrel.Mac/Squirrel/SQRLDirectoryManager.m:30
Illegal instruction: 4
Can someone please guide me how to work on Atom in Mac as I am using Atom editor for the very first time.
Regards,
Shailender

Hi All,
I am now able to open Atom perfectly on my Mac, seems like the download got corrupted during the download.
Thanks,
Shailender


Your solution to Issue #208 is problematic

Sorry, just forbidding the usage of "-" in properties is very problematic. Some properties in K8s manifests (like ephemeral-storage) contains this char and can't be set any more.
Our K8s cluster expects it's defined.

Having these characters cause the Helm chart installation to fail with, for example:
helm install --dependency-update fruits-app target/helm/kubernetes/fruits-app
Error: INSTALLATION FAILED: parse error at (fruits-app/templates/servicebinding.yaml:1): bad character U+002D '-'

How these characters can cause installation issues and on the other hand, be mandatory in other scenarios?

I think this is a different use case: I want to define a property in a kubernetes manifest (which is run by kubectl) but the installation issue arises in a part which is interpreted by helm (or Go).
I'm using something like
quarkus.helm.values.5.property=postgresql.primary.resources.limits.ephemeral-storage
quarkus.helm.values.5.value=500Mi

to populate my values.yaml, the so generated part is just copied over by my helm template but neither parsed nor interpreted.
Perhaps it would be sufficient to allow values of properties with a dash if they are defined under
quarkus.helm.value...

Sorry, I wasn't aware of this possibility! Thanks for your help and your patience!

Glad it helped!


[BUG] Export of role template fails when having 2 roles with the same name

Is there an existing issue for this?

[X] I have searched the existing issues

What version of the Terraform exporter are you using?
0.1.0-beta
What version of the Terraform CLI are you using?
1.9.9
What type of issue are you facing
bug report
Describe the bug
In the subaccount I try to export there are 2 role-templates with the same name (token_exchange) but with different Application Identifier (launchpad-dt-approuter!t3664, xsuaa!b120249). When running the export I get the following error:
´´´´
Error: Duplicate import configuration for "btp_subaccount_role.token_exchange"
│
│   on btp_subaccount_role_import.tf line 374, in import:
│  374:                                 to = btp_subaccount_role.token_exchange
│
│ An import block for the resource "btp_subaccount_role.token_exchange" was already declared at btp_subaccount_role_import.tf:369,1-7. A
│ resource can have only one import block.
╵
╷
│ Error: Duplicate import configuration for "btp_subaccount_role.token_exchange"
│
│   on btp_subaccount_role_import.tf line 374, in import:
│  374:                                 to = btp_subaccount_role.token_exchange
│
│ An import block for the resource "btp_subaccount_role.token_exchange" was already declared at btp_subaccount_role_import.tf:369,1-7. A
│ resource can have only one import block.
´´´´

Expected Behavior
It should be possible to import 2 role_templates with the same name but with different Application Identifier
Steps To Reproduce
btptf export
btptf export-by-json
removing of the double entries in the json files doesn't solve the problem
User's Role Collections
Global Account Admin, Subaccount Admin
Add screenshots to help explain your problem
No response
Additional context
No response

Thanks for bringing this up. We will be looking into this.


Steam Deck Config Defaults

config - default to full screen 1280x800

[x] openmw
[x] warzone2100
[x] openxcom
[x] openxcom-oxce
[x] jedi knight & academy
[x] ecwolf
[x] RealRTCW
[x] Good-Robot
[x] dhewm3
[x] OpenLoco
[x] iortcw
[x] ut99

automatic resolution

[x] Quake
[x] Quake II
[x] Quake III
[x] ET Legacy
[x] gzdoom
[x] dosbox-staging
[x] RetroArch
[x] Arx Libertatis
[x] scummvm
[x] Classic-RBDOOM-3-BFG
[x] RBDOOM-3-BFG
[x] Source SDK 2013
[x] augustus
[x] julius
[x] NBlood
[x] raze
[x] dxx-rebirth
[x] openrct2
[x] fs2open
[x] Hammer of Thyrion
[x] bstone
[x] rottexpr
[x] Cortex Command Community Project Source
[x] OpenGothic
[x] doomsday
[x] 1oom - forced to original resolution
[x] Perimeter
[x] OpenXRay
[x] engge

manual required

[x] avp
[x] REminiscence / REinforced
[x] unrealgold
[x] Daikatana
[x] BuildGDX

I've added default configs for games to help with resolution that I tested that needed it where possible. Feel free to comment here if there's any other games that could use some additional work.


Calculate polygon triangles manually

ShapeDrawer spends half the time in computeTriangles. So we should do that manually once instead of letting ShapeDrawer calculate it every time.

Implemented in c5786f4

Implemented in c5786f4


why are MP:0003122 maternal imprinting and MP:0003123 paternal imprinting phenotypes?

Why are MP:0003122 maternal imprinting and MP:0003123 paternal imprinting phenotypes? Maternal and paternal imprinting is supposed to happen! If it doesn't, you're in big trouble.
So it should really be:
MP:0003122 abnormal maternal imprinting
MP:0003123 abnormal paternal imprinting
as children of
MP:0003787 abnormal imprinting
with the definitions updated accordingly.
Or am I missing something?

maternal and paternal imprinting were added back when MGI decided not to have maternal and paternal versions of alleles. These are meant to indicate that the phenotypes of the mouse vary with maternal/paternal inheritance. Really these shouldn't be phenotypes but we stuck this in to be able to find these types of genotypes. Much like 'no phenotypic analysis' it would be good to come up with a different solution and obsolete these terms.

removed the parent 'abnormal epigenetic regulation of gene expression'

genetic imprinting is defined as abnormal but the label could be interpreted as normal and this has an abnormal imprinting child term. I believe this was intended to be a normal

Modified textual def of genetic imprinting to "the process that modulates the frequency, rate or extent of the chemical reactions and pathways involving macromolecules by a mechanism that is mediated by DNA, is mitotically or meiotically heritable, or is stably self-propagated in the cytoplasm of a resting cell, and does not entail a change in DNA sequence" to match the term label better


allow for remote themes

This will fix #8 by changing to the source directory before running jekyll. This was failing because remote themes needs to be inside of the source directory to work even though the jekyll command without remote themes works as expected

I have check this with genomics-workshop, docker-introduction, and r-novice-gapminder. It seems to work for all of them :tada:


Reformat scratch.py  indentation

The file used a mixture of tabs and 4spaces for indentation which made editing it a bit tricky.  I've reformatted it so that it just uses multiples of 4 spaces as indentation

awesome, thanks!


Project

Hello.
I would like to know if UMod is stopped or not.
Thanks you.

Sorry for the late reply.
I didn't see my notifications before.
UMod as an Unreal game is stopped not the idea of the project.
The project is coming back however it takes a lot more time than I expected.
Also I'm student now, which means I have less time than before for own projects.
Anyways if you are interested in knowing what were the reasons of the shutdown, well it is related to the support UE4 have when it comes Modability in the way I expect for this project, the Unix based platforms support of Unreal, and other various components.
Indeed I started 2 years ago a replacement Engine designed for this project.
If you are interested in helping the new project going through, I suggest you take a look at the new github organisation for it : https://github.com/BlockProject3D
I will update the README of the repository as well.


added directory support for cli

I added directory support for cli because we use a lot of file in the same subdirectory (eg: src/ and want to make a single file like application.css and application.min.css) and we felt us naked without this =) If I added this then I think send a pull request.
If the given "file" is a directory
then :syntax option will be skipped and guessing from file extension.
Example: sass src/ assets/application.css
I know not a perfect and nice additional but works. If i will have time (and someone else don't make it) I make a better solution. If this solution is not ok then please tell me what is the problem.
Thanks: Balazs Nadasdi

this command creates separated files
$ sass --update src:assets
$ ls assets
one.css   three.css two.css
and this patch make just one file (or one output stream to stdout)
$ sass src assets/app.css
$ ls assets
app.css
Nothing more =) So i don't need to merge these files with an other app or i don't need to make a main.sass what contains only @imports's

Use partials.

Oh yeah before i modified this our base.sass was 63 lines thats only @import's
@import 'page'
@import 'main'
@import 'editor'
@import 'enterpage'
@import 'galleries'
@import 'images'
@import 'members'
@import 'messages'
And i we created a new file we need to add this it to somewhere. =/ So partial is NOT a nice solution for it. =) because a little annoying.

Handling many imports has been discussed to death elsewhere. The upshot is that we don't support it because it makes the ordering of the imports unclear, but if you really want it you can define a custom importer.

Thx =)


[iOS] Null reference exception with view change

I get the following error when I use the command to change the view.  This is similar to issue #44.
{System.NullReferenceException: Object reference not set to an instance of an object
  at TouchEffect.iOS.TouchUITapGestureRecognizer.HandleTouch (TouchEffect.Enums.TouchStatus status) [0x0004d] in C:\GitHub\TouchEffect\TouchEffect.iOS\PlatformTouchEff.cs:153
  at TouchEffect.iOS.TouchUITapGestureRecognizer.TouchesEnded (Foundation.NSSet touches, UIKit.UIEvent evt) [0x00012] in C:\GitHub\TouchEffect\TouchEffect.iOS\PlatformTouchEff.cs:85
  at (wrapper managed-to-native) UIKit.UIApplication.UIApplicationMain(int,string[],intptr,intptr)
  at UIKit.UIApplication.Main (System.String[] args, System.IntPtr principal, System.IntPtr delegate) [0x00005] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/Xamarin.iOS/UIKit/UIApplication.cs:86
  at UIKit.UIApplication.Main (System.String[] args, System.String principalClassName, System.String delegateClassName) [0x0000e] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/Xamarin.iOS/UIKit/UIApplication.cs:65
  at App.iOS.Application.Main (System.String[] args) [0x00001] in C:\GitHub\TouchEffect\App\App.iOS\Main.cs:17 }	System.NullReferenceException

See example here

A have the same bug with last version

A have the same bug with last version if I use toucheff:TouchEff.Command

Let me check...

@Brayanhenaor cannot reproduce on iOS.
Can you please build a sample?

I have the same bug with last version if I use toucheff:TouchEff.Command

provide a sample please if you want to get this bug fixed. Unfortunately, I cannot reproduce it

I have the same bug with last version if I use toucheff:TouchEff.Command

provide a sample please if you want to get this bug fixed. Unfortunately, I cannot reproduce it

her a sample
touch:TouchEff.Command="{Binding GoToMenuCommand}"
touch:TouchEff.IsAvailable="True"
touch:TouchEff.PressedBackgroundColor="Black"
touch:TouchEff.PressedOpacity="8"
touch:TouchEff.PressedScale="2"
touch:TouchEff.RegularAnimationDuration="300"
touch:TouchEff.RippleCount="1"

I have the same bug with last version if I use toucheff:TouchEff.Command

provide a sample please if you want to get this bug fixed. Unfortunately, I cannot reproduce it

her a sample
touch:TouchEff.Command="{Binding GoToMenuCommand}"
touch:TouchEff.IsAvailable="True"
touch:TouchEff.PressedBackgroundColor="Black"
touch:TouchEff.PressedOpacity="8"
touch:TouchEff.PressedScale="2"
touch:TouchEff.RegularAnimationDuration="300"
touch:TouchEff.RippleCount="1"

Unfortunately, it's not a sample. That's just several lines of xaml markup.
What kind of view should I apply it to? (ContentView, Label, Frame, StackLayoyt ... ?)

Can you build a small sample project where I can reproduce this issue?) Thanks

Are you sure you update TouchView nuget in both iOS and netstandard projects?

BTW, I have a couple of remarks about it:
touch:TouchEff.Command="{Binding GoToMenuCommand}"
touch:TouchEff.IsAvailable="True"
touch:TouchEff.PressedBackgroundColor="Black"
touch:TouchEff.PressedOpacity="8"
touch:TouchEff.PressedScale="2"
touch:TouchEff.RegularAnimationDuration="300"
touch:TouchEff.RippleCount="1"

touch:TouchEff.IsAvailable="True" This is the default value. So you can remove this line.
touch:TouchEff.PressedOpacity="8" Maximum opacity is 1. Wha does 8 mean? Maybe you wanted to set "0.8"?

Well, please try latest version (but make sure you installed the LATEST version to both iOS and Netstandard projects). Perhaps I fixed this issue.


Support for multi-namespace application

Issues we expect the ongoing effort to enable defining an appsub that is able to apply resources on various namespaces to solve:

Having one appsub that retrieves other appsubs from a git repository - this way OCM can define one appsub for essential configurations and the other appsubs would be configured within the git repo (e.g., for CNV, CNF).

Support appsubs that apply resources on various namespaces - OCM currently seems to support  appsubs that define resources in a namespace that is associated with it, the default namespace and non-namespaced resources. But we need to support CNF's performance-addon-operator that applies non-namespaced resources as well as resources in the operator's namespace (openshift-performance-addon) and in the openshift-marketplace namespace.

@jnpacker fyi

Agreed, we are pushing hard to enable this capability, but we are having trouble doing it securely. We believe we have a path now, that we should confirm this week, and if so, we will be able to address multi namespace

@rokej can you duplicate this to your current issue if you have one, otherwise please update status here going forward.

This was implemented. Tracking with https://github.com/open-cluster-management/backlog/issues/1295.


Button BG Color Modal Problem

Hi! I am using Semantic UI 2.0.2 and I ran into a problem having a button trigger a modal. I haven't built my modal yet - just using a test one but I've already run into a problem:
When I set the button to trigger the modal everything works fine, except when I close the modal the button then clicks back to its "hover" background color (a slightly darker grey) and then when I click somewhere else on the page it goes back to normal.
See below:
https://jsfiddle.net/n3cnrnrt/
This isn't quite the latest version of Semantic UI that I pulled in for the jsfiddle but it's the same problem I'm having with 2.0.2.
Any ideas?
Thanks!
Andrew
Update: When my button is created with <button> I get this problem but when I create it with a <div> like on the Semantic UI modal page it works fine. So this is probably still a bug because I think it should be able to work both ways.

This is an accessibility "feature" of <button> tags and pretty annoying. Buttons will retain focus after being clicked. The same issue will also occur in all the button docs examples.


Generate Snapshot files for TXIDs

Type of change

New feature

Description

Add reusable functions for creating and reading the snapshot files. This includes the encoding/decoding the information during creation/loading of snapshot files, and computing the hash while exporting the information. These functions are expected to be used by various components that would generate or consume the snapshot files, such as blockstorage and statedb.

Generate snapshot files for TXIDs by leveraging the above mentioned functions

FAB-17870
Additional details
For ease of review, the code is split into two commits but still submitted as a single PR, so as to be able to see the intended use of the reusable functions.

Just one nit, so I'll go ahead and merge.


Toggle mobile view button to see the documentation examples

I just want to suggest something i think is helpful, for the examples on bootstrap 4 documentations we need a mobile phone or tablette toggle button to see what will be the example on mobile, now is there just html comment that say the box is 100% width on desktop but on mobile is 50% width, when i want to see what will be happen i resize the browser window but the example i want scroll to top or bottom and i just scroll down to find that example,
Would you please add toggle mobile button side by side every exemple?
Thank you (edited)

in terms of feasibility, that would likely require each example to be in an <iframe>, and scripting to then dynamically change the dimensions of the <iframe> (and potentially the zoom level of the embedded document too)

I would suggest different buttons for all 5 responsive breakpoints.

We're not going to do a mobile toggle—the docs are responsive for a reason. We're also definitely not going to add five buttons or options for all the breakpoints. Thanks for the suggestion, though!

the docs are responsive for a reason

Responsible yes, useful no. For example grid offsetting-columns, that page is so long that resizing the window scrolls the content out of view very fast, making it almost impossible to see the differences between different breakpoints.

Thnak you @mdo for your replay, i dont said that we want button o all the pages and all the breakpoints, there is some example definitely have a toggle button to see it not resize a big mega long page to see what will be on the mobile, you can suggest this feature on github and see what user want, and i am sure 100% any person will love this button.
Bootstrap 4 is a big project not just that a simple website i can just resize the window or see it on my phone to see the change you know Bootstrap not a website it is a mega project.
Thank you.


constrainChildren causes collapsed group resize to mess up contained cells

First of all, thanks for open sourcing a wonderful project! It's amazing and I love the ability to add plugins :)
Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
if mxGraph.constrainChildren is enabled, resizing a collapsed group causes the contained cells to be squashed and moved around. See gif.
I realize this might not matter to you because it doesn't look like constrainChildren is used much in diagrams.net, but thought I would submit the issue anyway.
To Reproduce
Steps to reproduce the behavior:
See gif below with exact steps.

enable constrainChildren on mxGraph
collapse a group that has content inside it
resize the group
expand the group
notice that group contained cells have been moved and squashed

Expected behavior
The group's contained cells should not be moved or squashed.
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 20.8.3

Desktop (please complete the following information):

Windows
Chrome
Version 108.0.5359.125 (Official Build) (64-bit) (cohort: Stable)

Fix in 21.8.0.


Create SECURITY.md007

38qopLNuY5QZ9PRf2HA6Ubso4zFiqFS6hv
Thank you for contributing! Please confirm this pull request meets the following requirements:

[ ] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[ ] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288)
[ ] Content (and my changes are in index.md)

Please replace this line with an explanation of why you think these changes should be made.

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[x] My folder contains a *.png image (if applicable) and index.md
[ ] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please replace this line with an explanation of why you think this topic or collection should be curated.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

contributor_covenant-release.zip
Vertical screen

38qopLNuY5QZ9PRf2HA6Ubso4zFiqFS6hv

38qopLNuY5QZ9PRf2HA6Ubso4zFiqFS6hv

New version has

Lisa Budzinski
On Thu, Jul 4, 2019, 8:58 AM Justin Kenyon<EMAIL_ADDRESS>wrote:

Closed #841 https://github.com/github/explore/pull/841.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/github/explore/pull/841?email_source=notifications&email_token=AHLLGCRDKB3IDQHHOTK3Y33P5XXXBA5CNFSM4H4MYM4KYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOSKUKYAI#event-2460527617,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHLLGCRKR3ZLM65EAHCJVKLP5XXXBANCNFSM4H4MYM4A
.

Lisa Budzinski
On Wed, Jul 10, 2019, 9:29 PM Lisa Budzinski<EMAIL_ADDRESS>wrote:

Lisa Budzinski
On Thu, Jul 4, 2019, 8:58 AM Justin Kenyon<EMAIL_ADDRESS>wrote:

Closed #841 https://github.com/github/explore/pull/841.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<
https://github.com/github/explore/pull/841?email_source=notifications&email_token=AHLLGCRDKB3IDQHHOTK3Y33P5XXXBA5CNFSM4H4MYM4KYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOSKUKYAI#event-2460527617
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AHLLGCRKR3ZLM65EAHCJVKLP5XXXBANCNFSM4H4MYM4A
.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/github/explore/pull/841?email_source=notifications&email_token=AHLLGCWFCMSPYLPMQFSXSX3P62EI7A5CNFSM4H4MYM4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZVGSBY#issuecomment-510290183,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHLLGCVQZ3KB7SNL5LXGZV3P62EI7ANCNFSM4H4MYM4A
.


Contribute Solution for LeetCode October Day 16

Day 16:
Contribute a solution for this day's LeetCode problem.
Rules:

Comment on this issue if you're willing to contribute.
Each user can contribute a maximum of 1 solution in Java, C++, or Python.
Each user can contribute a maximum of 2 solutions in other languages like JavaScript, Go, Ruby, etc.
Please ensure that no one has already commented for the language you're planning to contribute in.

Good luck and happy coding! :tada: :rocket:

Hi @arya2004, I would like to contribute in Java for the same.

Hi @arya2004  , I would like to contribute in Cpp assign me

Hi @arya2004 , I would like to contribute in python assign me

Hi @arya2004 I have raised an PR for the same


Only allow pair (Maybe FunctionTypeDeclaration, FunctionDeclaration) as Statement ?

Since a FunctionTypeDeclaration without a corresponding FunctionDeclaration is not legal (syntax?) in Elm, would it be useful to change:
type Statement
    = ...
    | FunctionTypeDeclaration Name Type
    | FunctionDeclaration Name (List Expression) Expression
    | ...

to:
type alias FunctionTypeDeclaration = FunctionTypeDeclaration Name Type
type alias FunctionDeclaration = FunctionDeclaration Name (List Expression) Expression
type Statement
    = ...
    | FunctionDefinition
           (Maybe FunctionTypeDeclaration, FunctionDeclaration)
    | ...

with corresponding tweak to parser rules?

There are countless cases like this. There needs to be another layer on top
of the parser that outputs a representation that disallows invalid syntax.
On 16 Jun 2017 5:48 p.m., "desmond-dsouza"<EMAIL_ADDRESS>wrote:

Since a FunctionTypeDeclaration without a corresponding
FunctionDeclaration is not legal (syntax?) in Elm, would it be useful to
change:
type Statement
= ...
| FunctionTypeDeclaration Name Type
| FunctionDeclaration Name (List Expression) Expression
| ...
to:
type alias FunctionTypeDeclaration = FunctionTypeDeclaration Name Type
type alias FunctionDeclaration = FunctionDeclaration Name (List Expression) Expression
type Statement
= ...
| FunctionDefinition
(Maybe FunctionTypeDeclaration, FunctionDeclaration)
| ...
with corresponding tweak to parser rules?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Bogdanp/elm-ast/issues/66, or mute the thread
https://github.com/notifications/unsubscribe-auth/AL3sFOd9HPrCxJ77UXjvnDBvkayb-u_fks5sEpXKgaJpZM4N8jaL
.

ok, thanks for the feedback

Closing since it's very disruptive and it yields little value


test getting supported assets and their payment methods

Superseded by https://github.com/haveno-dex/haveno-ts/pull/103


Add support for .globalconfig

Hey there! I have a .globalconfig file in the root of my projects repo but changes to it don't seem to get picked up by the LSP. In order for the changes to take effect, I have to restart VS Code.
I have tried adding the config file to Directory.Build.props, but the LSP still doesn't detect the changes and MSBuild complains about there being duplicate files added.
<ItemGroup Label="GlobalAnalyzers">
    <GlobalAnalyzerConfigFiles Include="$([System.IO.Path]::Combine($(MSBuildThisFileDirectory), '.globalconfig'))" />
</ItemGroup>

It is likely we are not creating file watchers for global configs.


Some typos found...

Delete it or go to Transifex ?

@kant It seems the translation files don't support these characters 😭. Not sure what the fix is but can't merge this as-is.


Update command

Regex in golang
stack overflow

By default . does not match newlines. To change that, use the s flag:

r := regexp.MustCompile(`(?s)variable (.*)(description[ ]*= "(.)*")?`)

Links
How to overwrite file content in golang


Reference to something two directories higher is not working

Hi,
I have a strange issue with swagger-tools 0.10.1 . I'm using lots of references and especially lots of references from external files. Now I would like to use a reference from a file which is two directories up in the directory structure than the original file.
This is the situation:
tree
.
├── definitions
├── definitions.yaml
└── directory1
    └── directory2
        └── api-definition.yaml

$ cat definitions.yaml
definitions:
  test:
      type: string

$ cat directory1/directory2/api-definition.yaml
swagger: "2.0"

info:
  version: "2.3.0"
  title: test-api
  license:
    name: "nope"
    url: https://nope/

basePath: "/test/v1"

schemes:
  - https

consumes:
  - "application/json"
produces:
  - "application/json"

paths:
  '/test':
    #SOL003 location: 5.4.2
    post:
      description: "Test"
      parameters:
        - name: test
          description: test
          in: body
          required: true
          schema:
            $ref: "../../definitions.yaml#/definitions/test"
      responses:
        200:
          description: OK
          schema:
            type: string

$ swagger-tools  validate -v directory1/directory2/api-definition.yaml

Validation Details:

  Swagger Version: 2.0
  Swagger file: directory1/directory2/api-definition.yaml

API Errors:

  #/paths/~1test/post/parameters/0/schema/$ref: Reference could not be resolved: ../../definitions.yaml#/definitions/test

1 error and 0 warnings

Wild.  I'll have to take a peek.


Loading icons have visual artifact on Firefox

Version
@nuxt/ui: 2.8.0,
nuxt: 3.7.1
Reproduction Link
https://ui.nuxt.com/forms/input#loading, on Firefox
See this :

Steps to reproduce
Only use the default loading icon with Firefox
What is Expected?
Spin the loading icon, without artifact.
What is actually happening?
We can see a square around the loading spinner, spinning with the icon.
This issue is only for Firefox. This is working correctly on Chromium based browser. (maybe Safari need to be tested too)

Possibly related to https://bugzilla.mozilla.org/show_bug.cgi?id=1671784. It has been closed 12 days ago, maybe not released yet.


How can I uninstall VScode-arm?

How can I uninstallVScode-arm? is there any quick way to uninstall?
Thanks!

If installed from the .deb file, you should be able to apt-get uninstall code